Skip to content

Fix #2947#2966

Closed
cspotcode wants to merge 3 commits intocasey:masterfrom
cspotcode:fix-2947
Closed

Fix #2947#2966
cspotcode wants to merge 3 commits intocasey:masterfrom
cspotcode:fix-2947

Conversation

@cspotcode
Copy link
Copy Markdown
Contributor

On Windows, explicitly set $PATH of spawned child processes so that executables resolve in $PATH order, and C:\Windows\System32 doesn't override it.

Fixes #2947

@casey
Copy link
Copy Markdown
Owner

casey commented Mar 21, 2026

This is resolved in in #3160, which adds a new method CommandExt::resolve, which respects both PATH and PATHEXT on Windows, and uses it wherever commands are launched.

@casey casey closed this Mar 21, 2026
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.

just prioritizes System32 over $Path; is impossible to put bash on $Path if you have WSL2 installed

2 participants