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
shutil.which() should preserve path case #61197
Comments
Now which lowercase found directory name and extension on Windows. >>> shutil.which("python")
'c:\\python33\\python.exe' Proposed patch preserve case. >>> shutil.which("python")
'C:\\Python33\\python.exe' Please test this on Windows. |
Somehow the patch is not showing for me. |
Oh, sorry. Here is a patch. |
New changeset d2db601a53b3 by Serhiy Storchaka in branch '3.3': New changeset 5faae2bdf1e0 by Serhiy Storchaka in branch 'default': |
Test broken on Windows because PATHEXT contains uppercased extension ".EXT". Proposed solutions:
|
New changeset 28282e4e9d04 by Serhiy Storchaka in branch '3.3': New changeset e8f40d4f497c by Serhiy Storchaka in branch 'default': |
I chose the first simplest variant. |
Serhiy, test_pathext_checking still fails on many Windows buildbots: http://buildbot.python.org/all/waterfall?category=3.x.stable&category=3.x.unstable |
Thank you, Stefan. This relates to bpo-16957. |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: