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

triplet change in bash #5232

Open
1 task done
wsfulton opened this issue Feb 26, 2025 · 1 comment
Open
1 task done

triplet change in bash #5232

wsfulton opened this issue Feb 26, 2025 · 1 comment
Labels

Comments

@wsfulton
Copy link

Description / Steps to reproduce the issue

076c4a9 changes the environment variable from

MACHTYPE=x86_64-pc-msys

to

MACHTYPE=x86_64-pc-cygwin

We use this environment variable to determine whether we can pass / or - to Windows executables as sometimes / is expanded to a Windows path such as C:/msys64/nologo. So this breaks how we pass options to Windows executables. This is in a Mingw-w64 + msys2 environment. Is there another more correct way to detect when bash expands '/' that we should be using? Maybe this is something to do with Mingw-64 rather than msys, but if not, this seems to be a regression in the way bash is built for msys2.

Expected behavior

See above

Actual behavior

See above

Verification

Windows Version

MSYS_NT-10.0-20348

Are you willing to submit a PR?

No response

@wsfulton wsfulton added the bug label Feb 26, 2025
@lazka
Copy link
Member

lazka commented Mar 3, 2025

You can disable the conversion in all cases using export MSYS2_ARG_CONV_EXCL='*' see https://www.msys2.org/docs/filesystem-paths/#automatic-unix-windows-path-conversion

Or does this not help in your case?

Another way to check for our bash is to check if the "MSYSTEM" env var exists.

For more context, the change you linked is part of https://www.msys2.org/news/#2025-02-14-moving-msys2-closer-to-cygwin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants