[issue14599] Windows test_import failure thanks to ImportError.path
Roundup Robot devn...@psf.upfronthosting.co.za added the comment: New changeset 9e164b404983 by Brett Cannon in branch 'default': Issues #15169, #14599: Make PyImport_ExecCodeModuleWithPathnames() use http://hg.python.org/cpython/rev/9e164b404983 -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14599 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14599] Windows test_import failure thanks to ImportError.path
Brett Cannon br...@python.org added the comment: MAXPATHLEN no longer shows up in Python/import.c. -- dependencies: +Clear C code under PyImport_ExecCodeModuleObject() resolution: - fixed status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14599 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14599] Windows test_import failure thanks to ImportError.path
Brett Cannon br...@python.org added the comment: Thanks for the info, Roumen. It looks like there is a single use of MAXPATHLEN still in import.c which I hope to eliminate, so hopefully that will rectify this problem. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14599 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14599] Windows test_import failure thanks to ImportError.path
Roumen Petrov bugtr...@roumenpetrov.info added the comment:
The issue is related to MAXPATHLEN limit and how is implemented . One part of
request are already closed , another part wait . Some people port patches other
wrote that would not use very long path. I'm in the second group.
This buildbot use path with length 54 and since build tree is detected and to
this path code adds 'build'. For installed python code will use TEMP
environment variable.
The test are run in subdir test_python_${PID} and depending of PID the path is
with different length. Note this.
So try to reproduce with installed python and TEMP set to path like this
C:\12345678\12345678\12345678\12345678\12345678\12345678 and run test. The
limit of 260(-1) will be reached and FileNotFoundError will be raised.
The default value of TEMP is to short to trigger error.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14599
___
___
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14599] Windows test_import failure thanks to ImportError.path
Brett Cannon br...@python.org added the comment: It's not a problem, Stefan. I just happened to have already added the importlib.invalidate_caches() call to test_reprlib so I know that isn't the issue. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14599 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14599] Windows test_import failure thanks to ImportError.path
Brett Cannon br...@python.org added the comment: If you look at line 127 in importlib/_bootstrap.py you will see that it is an os.open() call to open the bytecode file for exclusive writing. I'm willing to bet the buildbot didn't have the directory writable or something and that triggered the issue. Regardless, this has nothing to do with the finder's cache. Antoine might have a little more insight since he wrote the atomic code initially. If not then this issue should be closed. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14599 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14599] Windows test_import failure thanks to ImportError.path
Roumen Petrov bugtr...@roumenpetrov.info added the comment: There is one long standing issue with length of the build path ... -- nosy: +rpetrov ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14599 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14599] Windows test_import failure thanks to ImportError.path
Brett Cannon br...@python.org added the comment: Roumen, what issue is that? Do you have an issue # you can share? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14599 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14599] Windows test_import failure thanks to ImportError.path
Stefan Krah stefan-use...@bytereef.org added the comment: Brett, I reopened this because you said earlier that the test_reprlib failure is due to a race condition where an invalidate_caches() call is needed. You're quite right of course that the new occurrence could be caused by something unrelated. I can't reproduce the failure on Windows 7, so it might be limited to XP. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14599 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14599] Windows test_import failure thanks to ImportError.path
Stefan Krah stefan-use...@bytereef.org added the comment: I think the issue mentioned in eae7cc54d28b just occurred again: http://www.python.org/dev/buildbot/all/builders/x86%20XP-4%203.x/builds/6807/steps/test/logs/stdio == ERROR: test_module (test.test_reprlib.LongReprTest) -- Traceback (most recent call last): File D:\cygwin\home\db3l\buildarea\3.x.bolen-windows\build\lib\test\test_reprlib.py, line 257, in test_module from areallylongpackageandmodulenametotestreprtruncation.areallylongpackageandmodulenametotestreprtruncation import areallylongpackageandmodulenametotestreprtruncation File frozen importlib._bootstrap, line 1333, in _handle_fromlist File frozen importlib._bootstrap, line 1286, in _find_and_load File frozen importlib._bootstrap, line 1253, in _find_and_load_unlocked File frozen importlib._bootstrap, line 432, in _check_name_wrapper File frozen importlib._bootstrap, line 776, in load_module File frozen importlib._bootstrap, line 757, in load_module File frozen importlib._bootstrap, line 408, in module_for_loader_wrapper File frozen importlib._bootstrap, line 633, in _load_module File frozen importlib._bootstrap, line 743, in get_code File frozen importlib._bootstrap, line 819, in set_data File frozen importlib._bootstrap, line 127, in _write_atomic FileNotFoundError: [Errno 2] No such file or directory -- -- nosy: +skrah resolution: fixed - status: closed - open ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14599 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14599] Windows test_import failure thanks to ImportError.path
Changes by Brett Cannon br...@python.org: -- title: Windows test_import failure - Windows test_import failure thanks to ImportError.path ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14599 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14599] Windows test_import failure thanks to ImportError.path
Roundup Robot devn...@psf.upfronthosting.co.za added the comment: New changeset 573010778eed by Brett Cannon in branch 'default': Issue #14599: Generalize a test for ImportError.path and add support http://hg.python.org/cpython/rev/573010778eed -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14599 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14599] Windows test_import failure thanks to ImportError.path
Roundup Robot devn...@psf.upfronthosting.co.za added the comment: New changeset 56aa4cda11a8 by Brett Cannon in branch 'default': Issue #14599: Support ImportError.path on AIX and HPUX when loading http://hg.python.org/cpython/rev/56aa4cda11a8 -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14599 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14599] Windows test_import failure thanks to ImportError.path
Changes by Brett Cannon br...@python.org: -- assignee: - brett.cannon resolution: - fixed status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14599 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
