traverseda
/
cpython
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Patch #595014: Cygwin tempfile patch 

Although Cygwin attempts to be as Posix compliant
as possible, it has difficulties unlinking open
files. This is not surprising given that Cygwin is
dependent on Win32 which in turn has this problem
itself.

The attached tempfile patch acknowledges this
Cygwin limitation. Without this patch, Cygwin
fails test_tempfile (i.e., test_has_no_name) as
follows:

$ ./python -E -tt ../Lib/test/regrtest.py -l test_tempfile
test_tempfile
test test_tempfile failed -- Traceback (most recent call last):
  File "/home/jt/src/PythonCvs/Lib/test/test_tempfile.py", line 689, in test_has_no_name
    self.failOnException("rmdir", ei)
  File "/home/jt/src/PythonCvs/Lib/test/test_tempfile.py", line 33, in failOnException
    self.fail("%s raised %s: %s" % (what, ei[0], ei[1]))
  File "/home/jt/src/PythonCvs/Lib/unittest.py", line 260, in fail
    raise self.failureException, msg
AssertionError: rmdir raised exceptions.OSError: [Errno 90] Directory not empty: '/mnt/c/DOCUME~1/jatis/LOCALS~1/Temp/tmpM_z8nj'
This commit is contained in:
Jason Tishler 2002-08-14 15:10:09 +00:00
parent 09707e3637
commit 80c02af345
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Lib/tempfile.py
View File

@ -410,9 +410,9 @@ def NamedTemporaryFile(mode='w+b', bufsize=-1, suffix="",
file = _os.fdopen(fd, mode, bufsize) file = _os.fdopen(fd, mode, bufsize)
return _TemporaryFileWrapper(file, name) return _TemporaryFileWrapper(file, name)
if _os.name != 'posix': if _os.name != 'posix' or _os.sys.platform == 'cygwin':
# On non-POSIX systems, assume that we cannot unlink a file while # On non-POSIX and Cygwin systems, assume that we cannot unlink a file
# it is open. # while it is open.
TemporaryFile = NamedTemporaryFile TemporaryFile = NamedTemporaryFile
else: else: