[issue1318] Remove os.tmpnam() and os.tempnam() [patch]

2007-10-25 Thread Christian Heimes
Christian Heimes added the comment: os.tmpfile() is the only method that has no duplicate in tempfile. I chose to keep it for this very reason. But you made good point, too. What do you think about renaming tmpfile to _tmpfile and make it available from the tempfile module as tempfile.tmpfile()?

[issue1318] Remove os.tmpnam() and os.tempnam()

2007-10-25 Thread Martin v. Löwis
Martin v. Löwis added the comment: os.tmpfile() is the only method that has no duplicate in tempfile. Why do you say that? tempfile.mkstemp() does essentially the same as os.tmpfile(). The promise of tempfile.mkstemp is also bogus for every OS except Windows. IIRC only Windows supports

[issue1318] Remove os.tmpnam() and os.tempnam()

2007-10-25 Thread Christian Heimes
Christian Heimes added the comment: Guido van Rossum wrote: Christian can you revise your patch to also remove os.tmpfile per Martin's request? Make sure unit tests and docs are fixed too. I can do that for you. But I still believe that os.tmpfile() works different than tempfile.mkstemp().

[issue1318] Remove os.tmpnam() and os.tempnam()

2007-10-25 Thread Guido van Rossum
Guido van Rossum added the comment: Christian can you revise your patch to also remove os.tmpfile per Martin's request? Make sure unit tests and docs are fixed too. __ Tracker [EMAIL PROTECTED] http://bugs.python.org/issue1318 __

[issue1318] Remove os.tmpnam() and os.tempnam()

2007-10-25 Thread Guido van Rossum
Guido van Rossum added the comment: 2007/10/25, Christian Heimes [EMAIL PROTECTED]: I can do that for you. But I still believe that os.tmpfile() works different than tempfile.mkstemp(). The former returns a file descriptor of a file w/o directory entry and the later a file in

[issue1318] Remove os.tmpnam() and os.tempnam()

2007-10-25 Thread Christian Heimes
Changes by Christian Heimes: Added file: http://bugs.python.org/file8612/py3k_remove_os_tmp.patch __ Tracker [EMAIL PROTECTED] http://bugs.python.org/issue1318 __ ___ Python-bugs-list

[issue1318] Remove os.tmpnam() and os.tempnam()

2007-10-25 Thread Guido van Rossum
Guido van Rossum added the comment: Committed revision 58657. Thanks! -- resolution: - accepted status: open - closed __ Tracker [EMAIL PROTECTED] http://bugs.python.org/issue1318 __

[issue1318] Remove os.tmpnam() and os.tempnam() [patch]

2007-10-23 Thread Martin v. Löwis
Martin v. Löwis added the comment: Removing them because there is a replacement already is a better reason than removing them because they give (bogus) warnings, so I'm -0 now. As you say, tempfile is not any better from a security point of view in the cases where tmpnam or tempnam would be