[Patches] [ python-Patches-888380 ] popen2.popen2() 'cmd' list
syntax docstring
SourceForge.net
noreply at sourceforge.net
Sat Jul 10 15:14:32 CEST 2004
Patches item #888380, was opened at 2004-01-31 18:44
Message generated for change (Comment added) made by jepler
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=888380&group_id=5470
Category: Documentation
Group: Python 2.3
Status: Open
Resolution: None
Priority: 5
Submitted By: Jeff Connelly aka shellreef (jeffconnelly)
Assigned to: Nobody/Anonymous (nobody)
Summary: popen2.popen2() 'cmd' list syntax docstring
Initial Comment:
This simple patch documents using popen2() with a list
as the first argument. Currently this feature is not
documented. It is good practice to use the list-popen2
as it avoids shell interpretation.
----------------------------------------------------------------------
Comment By: Jeff Epler (jepler)
Date: 2004-07-10 08:14
Message:
Logged In: YES
user_id=2772
Please resubmit the patch in context or unified diff format.
ed-format diffs generally won't apply when any other
changes have been made in that file in the meantime.
The same or similar documentation change should also be
applied to Popen3.__init__, Popen4.__init__ (which does't
exist yet), popen3(), and popoen4()
The line number in your version of the patch seems to
indicate that the change was made within the 'if
sys.platform[:3] == "win" ...:' block, which is where the
advice doesn't apply.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=888380&group_id=5470
More information about the Patches
mailing list