[issue21044] tempfile.TemporaryFile() shouldn't have a name attribute

2014-03-24 Thread Antoine Pietri
Antoine Pietri added the comment: I attached a patch for tarfile with eryksun's suggestion. -- keywords: +patch Added file: http://bugs.python.org/file34603/tarfile-fileobjname.diff ___ Python tracker _

[issue21044] tempfile.TemporaryFile() shouldn't have a name attribute

2014-03-23 Thread Martin Panter
Changes by Martin Panter : -- nosy: +vadmium ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pytho

[issue21044] tempfile.TemporaryFile() shouldn't have a name attribute

2014-03-23 Thread eryksun
eryksun added the comment: This name attribute is documented here: http://docs.python.org/3/library/io#io.FileIO.name 3.4 Source: http://hg.python.org/cpython/file/04f714765c13/Modules/_io/fileio.c#l432 In PY2, os.fdopen sets the name to ''. See the related issue 13781. Here's the workaround

[issue21044] tempfile.TemporaryFile() shouldn't have a name attribute

2014-03-23 Thread Antoine Pietri
Antoine Pietri added the comment: Alternatively, if the "name" attribute can't be removed, I propose the following diff for the tarfile module: -if name is None and hasattr(fileobj, "name"): +if name is None and hasattr(fileobj, "name") and isinstance(fileobj.name, str): :-) -- _

[issue21044] tempfile.TemporaryFile() shouldn't have a name attribute

2014-03-23 Thread Antoine Pietri
New submission from Antoine Pietri: The fact that tempfile.TemporaryFile() has a "name" integer attribute causes weird behavior when interacting with libraries that rely on this attribute being a valid string for file objects. For instance, it led to this exception with the "tarfile" module, w