Trent Nelson added the comment:
With Chris and Ben's comments taken into account, what's the best way to
handle this?
1. Change the test: if the user is not admin, assert os.tmpfile()
returns a permission denied OSError, otherwise, assert return value is
a current file.
2. Alter
Benjamin Peterson added the comment:
With Chris and Ben's comments taken into account, what's the best way to
handle this?
I think, given that is being removed, we can safely go with option 1.
__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue2232
Trent Nelson added the comment:
I agree. Following patch fixes the issue for me:
Index: test_os.py
===
--- test_os.py (revision 61260)
+++ test_os.py (working copy)
@@ -65,6 +65,44 @@
def test_tmpfile(self):
if not
Joseph Armbruster added the comment:
Tested patch against: http://svn.python.org/projects/python/trunk @ 61260
OS Name: Microsoft Windows XP Professional
OS Version:5.1.2600 Service Pack 2 Build 260
rt test_os
Deleting .pyc/.pyo files ...
(57, '.pyc deleted,',
Martin v. Löwis added the comment:
Thanks for the patch. Committed as r61264 and r61266.
--
nosy: +loewis
resolution: - accepted
status: open - closed
__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue2232
__
Christian Heimes added the comment:
Side note:
I've removed the methods from Python 3.0 about half a year ago. Code
should use the tempfile module anyway. Does any of the Python 2.6 stdlib
code use an os.tmp* method?
--
nosy: +tiran
__
Tracker [EMAIL
Benjamin Peterson added the comment:
According to grep, the only place where os.tmp* is referenced is in test_os.
--
nosy: +benjamin.peterson
__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue2232
__
