Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Doument POSIX path translation in MSYS2 #176

Open
mbland opened this issue May 29, 2017 · 1 comment
Open

Doument POSIX path translation in MSYS2 #176

mbland opened this issue May 29, 2017 · 1 comment
Assignees
Projects
Milestone

Comments

@mbland
Copy link
Owner

mbland commented May 29, 2017

Adding the following to go-core.bash would avoid some nasty surprises:

if [[ "$OSTYPE" == 'msys' ]]; then
  export MSYS_NO_PATHCONV='true'
  export MSYS2_ARG_CONV_EXCL='*'
fi

per: How to stop mingw and msys from mangling path names given at the command line? (Stack Overflow)

This may have ramifications for the ./go get command, though I'm not sure yet.

@mbland mbland self-assigned this May 29, 2017
mbland added a commit to mbland/custom-links that referenced this issue May 29, 2017
This was causing paths to get mangled for `./go test browser`, which
launches live-server. For more details, see:

mbland/go-script-bash#176
mbland added a commit to mbland/custom-links that referenced this issue May 29, 2017
Turns out setting `MSYS_NO_PATHCONV='true'` and
`MSYS2_ARG_CONV_EXCL='*'` at the top level was a bad idea, as it breaks
`npm install`. So mbland/go-script-bash#176 should be closed.
@mbland
Copy link
Owner Author

mbland commented May 29, 2017

Per mbland/custom-links#62, setting this at the top level actually breaks some programs, notably npm install. So rather than automatically disabling it, it should be better documented somewhere.

@mbland mbland changed the title Automatically disable POSIX path translation in MSYS2 Doument POSIX path translation in MSYS2 May 29, 2017
@mbland mbland modified the milestone: v1.8.0 Aug 27, 2017
@mbland mbland added this to Backlog in v1.8.0 Oct 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
v1.8.0
Backlog
Development

No branches or pull requests

1 participant