Amaury Forgeot d'Arc <amaur...@gmail.com> added the comment:
The test is the proof the the regression, and shows the change in
behavior and the purpose of the development.
There happens to be a test for os.popen3 (in Lib/test/test_popen2.py)
which passes a list; this one at least passes.
os.popen3() accepts a sequence, except that since 2.6, the first item is
the whole command string, and additional items are treated as additional
shell arguments!
Your patch is correct. A unit test could be:
w, r, e = os.popen3(['echo', 'hello'])
assert r.read() == 'hello\n'
And should be disabled on non-posix platforms.
----------
keywords: +needs review
nosy: +amaury.forgeotdarc
_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue5329>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
