bpo-37412: pythoninfo: add Windows long paths#14434
bpo-37412: pythoninfo: add Windows long paths#14434vstinner merged 1 commit intopython:masterfrom vstinner:pythoninfo_long_paths
Conversation
|
The code comes from @eryksun, so I added him as a co-author: |
|
@eryksun: Would you mind to review this change? I tested it manually on my Windows VM: Maybe it was the Python installer who proposed me to enable this Windows feature, I don't recall. |
|
|
Oh, there is also a RtlIsLongPathAwareProcessByManifest() function which may be interesting to check: I updated my PR to also call this function. Result on my Windows 10 VM: Oh, RtlIsLongPathAwareProcessByManifest is not function not available in ntdll on my VM? :-( |
|
Results on AppVeyor: Results on Azure Pipeline, win32 and win64 (same values): |
|
RtlIsLongPathAwareProcessByManifest may be interesting when I will backport this change to old Python versions. I'm trying to keep the pythoninfo.py code the same in all branches. |
I wouldn't worry about
|
On Windows, test.pythoninfo now checks if support for long paths is enabled using ntdll.RtlAreLongPathsEnabled() function. Co-Authored-By: Eryk Sun <eryksun@gmail.com>
Ok ok. Anyway, this info can be retrieved indirectly from the Python version. I rebased my PR and squashed commits. @eryksun: Would you mind to review the updated PR? |
|
I merged my PR. Thanks @eryksun for reviews! |
On Windows, test.pythoninfo now checks if support for long paths is enabled using ntdll.RtlAreLongPathsEnabled() function. Co-Authored-By: Eryk Sun <eryksun@gmail.com>
On Windows, test.pythoninfo now checks if support for long paths is enabled using ntdll.RtlAreLongPathsEnabled() function. Co-Authored-By: Eryk Sun <eryksun@gmail.com>
On Windows, test.pythoninfo now checks if support for long paths is
enabled using ntdll.RtlAreLongPathsEnabled() function.
Co-Authored-By: Eryk Sun eryksun@gmail.com
https://bugs.python.org/issue37412