Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

bpo-40592: shutil.which will not return None anymore if ; is the last char in PATHEXT #20088

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.

Sign up for GitHub

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

Merged
merged 15 commits into from
Oct 23, 2020
Merged

bpo-40592: shutil.which will not return None anymore if ; is the last char in PATHEXT #20088

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
2981e84
shutil.which will not return None anymore for empty str in PATHEXT
May 14, 2020
9df7b99
📜🤖 Added by blurb_it.
blurb-it[bot] May 14, 2020
6dd9574
Defaulted missing PATHEX env to cmd defaults
May 14, 2020
65b2847
Merge branch 'master' of github.com:python/cpython into fix-shutil-wh…
May 14, 2020
49284d9
Merge branch 'fix-shutil-which-windows' of github.com:peanut-lord/cpy…
May 14, 2020
9eb17bb
Changed os.chmod from 'S_IXUSR' to 'S_IXUSR | S_IWUSR'
May 15, 2020
e6a9c2c
Added comment to explain the _WIN_DEFAULT_PATHEXT properly
May 15, 2020
4966956
Updated change description
peanutlord May 19, 2020
cdb179e
Adjusted list comprehension to adhere line length
May 19, 2020
01efb42
Removed chown from unittest
May 19, 2020
fc87309
Empty PATHEXT () will now also be defaulted to _WIN_DEFAULT_PATHEXT
Oct 21, 2020
6af00e7
When no extension is found, we return a list with the command
Oct 21, 2020
ee618eb
Revert "When no extension is found, we return a list with the command"
Oct 21, 2020
77de5b8
Merge branch 'master' of github.com:python/cpython into fix-shutil-wh…
Oct 22, 2020
6c66dfc
Switched to from support to os_helper in unittest
Oct 22, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Revert "When no extension is found, we return a list with the command" 
This reverts commit 6af00e7.
  • Loading branch information
Christopher Marchfelder committed Oct 21, 2020
commit ee618eb819d1c984cc79b1fa1a1a8a05a26bda79
3 changes: 1 addition & 2 deletions 3 Lib/shutil.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1427,8 +1427,7 @@ def which(cmd, mode=os.F_OK | os.X_OK, path=None):
# This will allow us to short circuit when given "python.exe".
# If it does match, only test that one, otherwise we have to try
# others.
if not pathext or any(
cmd.lower().endswith(ext.lower()) for ext in pathext):
if any(cmd.lower().endswith(ext.lower()) for ext in pathext):
files = [cmd]
else:
files = [cmd + ext for ext in pathext]
Expand Down
Morty Proxy This is a proxified and sanitized view of the page, visit original site.