Issue 1384: Windows fix for inspect tests (original) (raw)

This issue has been migrated to GitHub: https://github.com/python/cpython/issues/45725

classification

process

Created on 2007-11-04 18:44 by christian.heimes, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Messages (6)
msg57105 - (view)	Author: Christian Heimes (christian.heimes) *	Date: 2007-11-04 18:44
The patch lower()s the file names on Windows. The tests break on my system because C:\\... != c:\\...
msg57139 - (view)	Author: Guido van Rossum (gvanrossum) *	Date: 2007-11-05 21:36
Shouldn't this use os.path.normcase() instead of testing for nt and using lower()?
msg57145 - (view)	Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) *	Date: 2007-11-05 23:12
A slightly modified patch, which uses os.path.normcase before comparing file names.
msg57146 - (view)	Author: Christian Heimes (christian.heimes) *	Date: 2007-11-06 00:17
Ah, os.path has a method for the job. I should have checked the module before reinventing the wheel ... For some unknown and mysterious reasons the inspect tests are passing again on my machine. I've purged the pyc files before I run the unpatched test suite. Should I apply the patch anyway?
msg57150 - (view)	Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) *	Date: 2007-11-06 00:39
Yes, the patch is needed. The problem arises when you run the python executable in different ways WITHOUT deleting the .pyc files. Example on my machine: Note that the exact path to run python is not the same: >cd C:\dev\python\py3k\PCbuild8 >del /s ..\*.pyc >c:\dev\python\py3k\PCbuild8\win32debug\python_d.exe -E -tt ../lib/test/regrtest.py -v test_inspect [test OK] >C:\dev\python\py3k\PCbuild8\win32debug\python_d.exe -E -tt ../lib/test/regrtest.py -v test_inspect [test FAILED] If I always use the same path the tests succeed.
msg57159 - (view)	Author: Christian Heimes (christian.heimes) *	Date: 2007-11-06 11:45
Applied to py3k in r58875. I'm doing a svnmerge later. Thanks again for your help, pal!