New submission from David Benjamin:
This is somewhat of a nitpick. os.rename's documentation says If dst is a
directory, OSError will be raised. On Unix, this isn't completely true. If the
source is a directory and the destination is an empty directory, it'll
overwrite the former
David Benjamin added the comment:
Well, the theoretical deadlock's just if stdin is also a pipe, right? I think
there isn't be a difference between communicate and stdout.read if only stdout
is a pipe. Though it might be worth passing DEVNULL to stdin instead of
inheriting, just to be tidy
David Benjamin added the comment:
Yes, communicate is needed if you have multiple pipes and need to be careful
about both ends doing a blocking reads/writes on different ones. There's only
one pipe here. Eh, whatever. If you guys really want to use communicate, I
don't really care
New submission from David Benjamin:
The implementation of platform.architecture shells out to the file command. It
tries to escape quotes by replacing with \, but that's not sufficient.
$ python3.2 -c 'import platform; platform.architecture(foo\\\; echo Hi there
/tmp/Z; echo \\\)' cat /tmp
David Benjamin david...@mit.edu added the comment:
I don't think that patch works. Consider a dict subclass which has overridden
update. Or perhaps a list subclass which has overridden addition. It would be
quite poor if Python's behavior here w.r.t. which overrides are followed
switched
Changes by David Benjamin david...@mit.edu:
--
nosy: +davidben
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9334
___
___
Python-bugs-list mailing