Roundup Robot added the comment:
New changeset 3417a95df7e2 by Andrew Kuchling in branch 'default':
#6490: Expand documentation for os.popen().
http://hg.python.org/cpython/rev/3417a95df7e2
--
nosy: +python-dev
___
Python tracker
A.M. Kuchling added the comment:
Thanks for your patch!
--
resolution: - fixed
stage: needs patch - committed/rejected
status: open - closed
title: os.popen documentation is probably wrong - os.popen documentation is
probably wrong
___
Python
Sam Kimbrel added the comment:
Updated the patch as per Martin's notes in msg203483.
--
nosy: +sam.kimbrel
Added file: http://bugs.python.org/file34896/6490-os-popen-docs.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6490
A.M. Kuchling added the comment:
I thought the discussion of the return code was rather complicated and tried
rewriting it; new patch attached. Is it an improvement? The new version also
specifies the function's parameters.
--
nosy: +akuchling
Added file:
Sam Kimbrel added the comment:
Yes, I think that wording works a lot better. Thanks for the touch-up.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6490
___
Martin Panter added the comment:
Please apply Neil Muller’s documentation patch. It is certainly better than the
current state.
If you want to improve it further, maybe get rid of the trailing comma, and
mention that the close() method returns the exit status encoded like the wait()
Martin Panter added the comment:
Also it would be good to document that it returns a text stream, not a binary
stream.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6490
___
Changes by Terry J. Reedy tjre...@udel.edu:
--
title: os.popen documentation in 2.6 is probably wrong - os.popen
documentation is probably wrong
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6490