➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: Fix test_tempfile: space in Win32 install path bug #811082
Type: Stage:
Components: Tests Versions: Python 2.3
process
Status: closed Resolution: accepted
Dependencies: Superseder:
Assigned To: loewis Nosy List: loewis, midramas
Priority: normal Keywords: patch

Created on 2003-09-26 17:59 by midramas, last changed 2022-04-10 16:11 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
patch_for_bug_811082_2ndtry.diff midramas, 2003-10-21 21:54 context diff
Messages (4)
msg44695 - (view) Author: Michael Bartl (midramas) Date: 2003-09-26 17:59 
Fix: 
 
- corrected arguments passed to spawnl function and 
added comment to explain tricky win32 behaviour 
 
Bug description: 
 
test_tempfile.py fails on XP (probably other 
platforms too) when Python is installed into \Program 
Files\Python2.3
msg44696 - (view) Author: Martin v. Löwis (loewis) * (Python committer) Date: 2003-10-18 18:09 
Logged In: YES 
user_id=21627

The patch is incorrect. On Unix, I get

"/home/martin/work/py2.4/python": can't open file
'"/home/martin/work/py2.4/Lib/test/tf_inherit_check.py"'
test test_tempfile failed -- Traceback (most recent call last):
  File "/home/martin/work/py2.4/Lib/test/test_tempfile.py",
line 304, in test_noinherit
    self.failIf(retval > 0, "child process reports failure")
  File "/home/martin/work/py2.4/Lib/unittest.py", line 288,
in failIf
    if expr: raise self.failureException, msg
AssertionError: child process reports failure
msg44697 - (view) Author: Michael Bartl (midramas) Date: 2003-10-21 21:59 
Logged In: YES 
user_id=31511

Stupid me! As written in the comment the fix only applies 
for windows so the OS should have been checked. Uploaded a 
new patch, which was tested on Linux & Windows XP.
msg44698 - (view) Author: Martin v. Löwis (loewis) * (Python committer) Date: 2003-10-23 15:56 
Logged In: YES 
user_id=21627

Thanks, committed as test_tempfile.py 1.16 and 1.14.8.3;
NEWS 1.831.4.65.
History
Date User Action Args
2022-04-10 16:11:24adminsetgithub: 39310
2003-09-26 17:59:21midramascreate