os.translateCommandsAndPaths

[Jarod42]

What does this PR do?

Improve os.translateCommandsAndPaths.
Fix msvc rule with %[path].
Fix xcode4 with %[path].

How does this PR change Premake's behavior?

For usage of '%[path]'

Anything else we should know?

Add any other context about your changes here.
Tested in my testing repo repo

Did you check all the boxes?

• Focus on a single fix or feature; remove any unrelated formatting or code changes
• Add unit tests showing fix or feature works; all tests pass
• Mention any related issues (put closes #XXXX in comment to auto-close issue when PR is merged)
• Follow our coding conventions
• Minimize the number of commits
• Align documentation to your changes

[redorav]

I've looked at the code, I see what it's trying to do, it would be good to have an example of how it changes the behavior. I've not used the %[path] syntax before, only the %{ }, is there something mentioned in the documentation about it?

[Jarod42]

Not mentioned in doc currently.

Planned to add it after this PR (in Token.md and prebuildcommands.md probably).Doc added.

Basically, it is for commands (prebuild/postbuild). to have correct relative path (as those command should be relative to project/solution and not premake5.lua)

[nickclark2016]

Can you provide samples for how the behavior changes?

[Jarod42]

@nickclark2016 :
There is 3 fixes:

• msvc supports %[].
• xcode supports %[]
• Fix %[] to supports relative paths.

-- from ./bar/premake5.lua, for (generator) project in ./foo
os.translateCommandsAndPaths("cmdtool %[../foo/path1] %[path2/]", './bar', './foo', 'osx')

returned wrongly
cmdtool "foo/path1" "bar/path2/"
and now
cmdtool "path1" "../bar/path2/"