Skip to content

Fix the PATH mangling in action.yml#2723

Merged
joerick merged 2 commits intomainfrom
path-mangling
Feb 4, 2026
Merged

Fix the PATH mangling in action.yml#2723
joerick merged 2 commits intomainfrom
path-mangling

Conversation

@joerick
Copy link
Contributor

@joerick joerick commented Jan 30, 2026

Let's put the uv on PATH question aside, and fix #2663.

This change ensures that the PATH entries added by Github when you request shell: bash don't make it into the pwsh environment. It also keeps the uv binary on PATH if installed via extras:.

Would love to get this fixed soon, then we can figure out the right approach regarding uv discovery.

Let's put the `uv` on PATH question aside, and fix #2663.

This change ensures that the PATH entries added by Github when you request `shell: bash` don't make it into the pwsh environment. It also keeps the uv binary on PATH if installed via `extras:`.

Would love to get this fixed soon, then we can figure out the right approach regarding `uv` discovery.
Cherry-picked from #2718 and modified so we can test the action.yml file
@joerick
Copy link
Contributor Author

joerick commented Jan 30, 2026


Examples test run

Branch: example-config-test---path-mangling-2026-01-30T17-03-28

Service Config Status
github examples/github-minimal.yml Build

Generated by bin/run_example_ci_configs.py examples/github-minimal.yml


@joerick joerick marked this pull request as ready for review January 30, 2026 17:14
@joerick joerick requested a review from henryiii February 1, 2026 09:43
@joerick
Copy link
Contributor Author

joerick commented Feb 1, 2026

how does this look to you @henryiii?

Copy link
Contributor

@henryiii henryiii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can start with this.

@joerick joerick merged commit 6088ea2 into main Feb 4, 2026
43 checks passed
@joerick joerick deleted the path-mangling branch February 4, 2026 09:23
kkraus14 added a commit to kkraus14/cuda-python that referenced this pull request Feb 19, 2026
cibuildwheel v3.3.x has a PATH mangling bug on Windows that causes
Git's /usr/bin/link.exe to shadow MSVC's linker (pypa/cibuildwheel#2663).
The fix (pypa/cibuildwheel#2723) is merged but not yet released.

Downgrade to v3.2.1 which does not have the bug, removing all the
workarounds: DISTUTILS_USE_SDK=1, CIBW_ENVIRONMENT_PASS_WINDOWS for
MSVC vars, and the link.exe rename hack.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

meson-python link.exe issue since 3.3.0 release

2 participants