[x] I have added documentation regarding my changes where necessary
[x] Any pre-existing tests continue to pass
[x] Additional tests were made covering my changes
Types of Changes
[x] Bugfix (change that fixes an issue)
[ ] New Feature (change that adds functionality)
[ ] Documentation Update (if none of the other choices apply)
Proposed Changes
After some more testing I've figured out that we only need to convert to cygpath and use : when modifying the PATH env variable, not any of the other environment variables. https://cygwin.com/cygwin-ug-net/setup-env.html.
The previous cygpath commit was applying that fix to all environment variables, but it turns out we only need to apply it to PATH. This MR changes hab to only apply cygpath changes for the PATH environment variable.
Addresses #41 by only calling cygpath when processing the
PATH
env var.Checklist
Types of Changes
Proposed Changes
After some more testing I've figured out that we only need to convert to cygpath and use
:
when modifying thePATH
env variable, not any of the other environment variables. https://cygwin.com/cygwin-ug-net/setup-env.html.If you run this in cygwin
It outputs
\\example\a\path;c:\temp;c:\other;\\server\share
which is expected.If you try to do the same for
PATH
, it does not work as expectedOutputs:
'c',
is incorrect and could cause issues. If you reset your bash and run this instead, it works as expected:The previous cygpath commit was applying that fix to all environment variables, but it turns out we only need to apply it to
PATH
. This MR changes hab to only apply cygpath changes for thePATH
environment variable.