GH-145273: warn when we can't find the standard library#145274
GH-145273: warn when we can't find the standard library#145274FFY00 wants to merge 7 commits intopython:mainfrom
Conversation
Signed-off-by: Filipe Laíns <lains@riseup.net>
|
The main issue with this is that we don't have a way to disable these warnings, which is resolved by GH-145277. |
Signed-off-by: Filipe Laíns <lains@riseup.net>
| if not stdlib_dir or not isdir(stdlib_dir): | ||
| hint = home_hint if home else f'sys.prefix is set to {prefix}, is this correct?' | ||
| warn('WARN: Could not find the standard library directory! ' + hint) | ||
| elif (not platstdlib_dir and not build_prefix) or not isdir(platstdlib_dir): |
There was a problem hiding this comment.
I fixed the tests, but unless we skip the platstdlib_dir check when running from source, the warning will show up on make, during the pybuilddir.txt generation.
I would prefer to drop the build_prefix check, and instead add -X pathconfig_warnings=0 to the -m sysconfig --generate-posix-vars call in the makefile.
| elif (not platstdlib_dir and not build_prefix) or not isdir(platstdlib_dir): | |
| elif not platstdlib_dir or not isdir(platstdlib_dir): |
Signed-off-by: Filipe Laíns <lains@riseup.net>
Signed-off-by: Filipe Laíns <lains@riseup.net>
|
I was thinking the symlink tests were failing on Windows because the This means the tests were failing because the symlink target did not exist, but in cpython/Lib/test/support/__init__.py Lines 1700 to 1703 in 06b0920 |
This reverts commit 3a67467. Signed-off-by: Filipe Laíns <lains@riseup.net>
Signed-off-by: Filipe Laíns <lains@riseup.net>
|
Ah, cpython/Lib/test/support/__init__.py Lines 1719 to 1721 in 06b0920 I am not familiar with the build on Windows, but looking at |
|
It's weird that it needs to set Anyway, the CI is still finishing, but |
Signed-off-by: Filipe Laíns <lains@riseup.net>
Uh oh!
There was an error while loading. Please reload this page.