8000 bpo-40592: shutil.which will not return None anymore if ; is the last char in PATHEXT by peanutlord · Pull Request #20088 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

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
Empty PATHEXT () will now also be defaulted to _WIN_DEFAULT_PATHEXT
  • Loading branch information
Christopher Marchfelder committed Oct 21, 2020
commit fc87309eae7b72ba19341f5b7b8d08a8546f21a1
6 changes: 3 additions & 3 deletions Lib/shutil.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1418,9 +1418,9 @@ def which(cmd, mode=os.F_OK | os.X_OK, path=None):
path.insert(0, curdir)

# PATHEXT is necessary to check on Windows.
pathext = [ext for ext in
os.environ.get("PATHEXT", _WIN_DEFAULT_PATHEXT).split(
os.pathsep) if ext]
pathext_source = os.getenv("PATHEXT") or _WIN_DEFAULT_PATHEXT
pathext = [ext for ext in pathext_source.split(os.pathsep) if ext]

if use_bytes:
pathext = [os.fsencode(ext) for ext in pathext]
# See if the given file matches any of the expected path extensions.
Expand Down
0