summaryrefslogtreecommitdiffstats
path: root/Modules
diff options
context:
space:
mode:
authorSteve Dower <steve.dower@python.org>2022-11-02 18:38:40 (GMT)
committerGitHub <noreply@github.com>2022-11-02 18:38:40 (GMT)
commit3d889dc0a0efa6fcbd984ece365c9ac08d2cd4a9 (patch)
tree48126e7aa2ca872aa5252d8c295bbc8b9db6d260 /Modules
parentf520d720f667c87f7b70ed86ea58d73892d6b969 (diff)
downloadcpython-3d889dc0a0efa6fcbd984ece365c9ac08d2cd4a9.zip
cpython-3d889dc0a0efa6fcbd984ece365c9ac08d2cd4a9.tar.gz
cpython-3d889dc0a0efa6fcbd984ece365c9ac08d2cd4a9.tar.bz2
gh-98790: When DLLs directory is missing on Windows, assume executable_dir contains PYD files instead (GH-98936)
Diffstat (limited to 'Modules')
-rw-r--r--Modules/getpath.py28
1 files changed, 21 insertions, 7 deletions
diff --git a/Modules/getpath.py b/Modules/getpath.py
index e3558bc..90a6473 100644
--- a/Modules/getpath.py
+++ b/Modules/getpath.py
@@ -579,15 +579,28 @@ else:
# Detect exec_prefix by searching from executable for the platstdlib_dir
if PLATSTDLIB_LANDMARK and not exec_prefix:
if executable_dir:
- exec_prefix = search_up(executable_dir, PLATSTDLIB_LANDMARK, test=isdir)
- if not exec_prefix:
- if EXEC_PREFIX:
- exec_prefix = EXEC_PREFIX
- if not isdir(joinpath(exec_prefix, PLATSTDLIB_LANDMARK)):
- warn('Could not find platform dependent libraries <exec_prefix>')
+ if os_name == 'nt':
+ # QUIRK: For compatibility and security, do not search for DLLs
+ # directory. The fallback below will cover it
+ exec_prefix = executable_dir
+ else:
+ exec_prefix = search_up(executable_dir, PLATSTDLIB_LANDMARK, test=isdir)
+ if not exec_prefix and EXEC_PREFIX:
+ exec_prefix = EXEC_PREFIX
+ if not exec_prefix or not isdir(joinpath(exec_prefix, PLATSTDLIB_LANDMARK)):
+ if os_name == 'nt':
+ # QUIRK: If DLLs is missing on Windows, don't warn, just assume
+ # that it's all the same as prefix.
+ # gh-98790: We set platstdlib_dir here to avoid adding "DLLs" into
+ # sys.path when it doesn't exist, which would give site-packages
+ # precedence over executable_dir, which is *probably* where our PYDs
+ # live. Ideally, whoever changes our layout will tell us what the
+ # layout is, but in the past this worked, so it should keep working.
+ platstdlib_dir = exec_prefix = prefix
else:
warn('Could not find platform dependent libraries <exec_prefix>')
+
# Fallback: assume exec_prefix == prefix
if not exec_prefix:
exec_prefix = prefix
@@ -689,7 +702,8 @@ elif not pythonpath_was_set:
pythonpath.append(platstdlib_dir)
if stdlib_dir:
pythonpath.append(stdlib_dir)
- pythonpath.append(executable_dir)
+ if executable_dir not in pythonpath:
+ pythonpath.append(executable_dir)
else:
if stdlib_dir:
pythonpath.append(stdlib_dir)