AuroraPipeline/premake
1
0
Fork 0
Code Issues 1 Pull Requests Projects Releases 1 Wiki Activity

Merge pull request #242 from Blizzard/fix-path-join

Browse Source 
Recognize more situations where '..' should not be trimmed.
This commit is contained in:
starkos 2015-09-16 18:40:49 -04:00
commit 847fc633ac
2 changed files with 22 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/host/path_join.c
View File

@ -67,11 +67,11 @@ int path_join(lua_State* L)
}
/* if I hit a segment I can't trim, bail out */
if (*start == '$') {
break;
}
if (start[0] == '.' && start[1] == '.' && start[2] == '\0'){
if (strcmp(start, "..") == 0 /* parent dir */
|| strcmp(start, ".") == 0 /* current dir */
|| strstr(start, "**") != NULL /* recursive wildcard */
|| strchr(start, '$') != NULL) /* property expansion */
{
break;
}

11
tests/base/test_path.lua
View File

@ -323,6 +323,17 @@
test.isequal("$(ProjectDir)/$(TargetName)/../../Bin", path.join("$(ProjectDir)/$(TargetName)", "../../Bin"))
end
function suite.join_keepsComplexInternalEnvVar()
test.isequal("$(ProjectDir)/myobj_$(Arch)/../../Bin", path.join("$(ProjectDir)/myobj_$(Arch)", "../../Bin"))
end
function suite.join_keepsRecursivePattern()
test.isequal("p1/**.lproj/../p2", path.join("p1/**.lproj", "../p2"))
end
function suite.join_noCombineSingleDot()
test.isequal("p1/./../p2", path.join("p1/.", "../p2"))
end
--