-
Notifications
You must be signed in to change notification settings - Fork 168
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
winpty not respecting non-path-munging env vars, or aliases #125
Comments
FWIW, the MSYS2 path conversion behavior isn't part of upstream winpty -- the MSYS2 winpty package adds it -- https://github.com/Alexpux/MSYS2-packages/blob/ec549593cd9874c801b3a95106d032d23b3ac0b8/winpty/0002-fix-path-conversion.patch. |
Related to #88. |
Ok, so sorry if i am a little dumb, but how do i fix it or turn it off, or who should i start poking to get it fixed or turned off? |
Maybe a workaround is to use upstream winpty instead of MSYS2's packaged winpty. Longer term, those two environment variables ( |
Aliases are not expected to work. They're part of your shell, and the shell won't expand an alias if it's an argument to a program. (In this case, the program is |
This gist is a workaround of this issue (and others): https://gist.github.com/mikeslattery/9b618669b63fe9dc984d5924bbb3aaee A shorter less robust version could be implemented in .bashrc:
|
I am using the latest version that comes with Git for Windows (git-bash).
MinGW 12:31:29 ~$ winpty --version winpty version 0.4.3 commit none
I have the following environment variables and aliases set:
The MSYS_NO_PATHCONV and MSYS2_ARG_CONV_EXCL are supposed to prevent path munging on both MSYS2 and Git-Bash, and they work fine, except when using winpty.
Examples:
You can see that with normal docker I can run
/bin/bash
, but with winpty I have to use//bin/bash
otherwise the path gets mucked up and changed to a windows directory.My aliases also do not seem to carry over:
The text was updated successfully, but these errors were encountered: