[issue2232] Current os.tmpfile() implementation requires admin privs on Vista/2k8.

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]> __

[issue2232] Current os.tmpfile() implementation requires admin privs on Vista/2k8.

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,',

[issue2232] Current os.tmpfile() implementation requires admin privs on Vista/2k8.

Trent Nelson added the comment: Er, errno being referred to in a comment in that last patch should be 13, not 12. Added file: http://bugs.python.org/file9617/test_os.py.2.patch __ Tracker <[EMAIL PROTECTED]>

[issue2232] Current os.tmpfile() implementation requires admin privs on Vista/2k8.

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

[issue2232] Current os.tmpfile() implementation requires admin privs on Vista/2k8.

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]> _

[issue2232] Current os.tmpfile() implementation requires admin privs on Vista/2k8.

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 posix_tmpfile

[issue2232] Current os.tmpfile() implementation requires admin privs on Vista/2k8.

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]> __

[issue2232] Current os.tmpfile() implementation requires admin privs on Vista/2k8.

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 PRO

[issue2232] Current os.tmpfile() implementation requires admin privs on Vista/2k8.

New submission from Trent Nelson: posix_tmpfile() needs to be reworked such that tmpfile() isn't used if MS_WINDOWS is defined, as it requires administrative privileges to run (it creates the file in the root directory) on Vista/2k8 (although there are reports of this not working on XP w/ 2.5: