[Patches] [ python-Patches-888380 ] popen2.popen2() 'cmd' list syntax docstring

[SourceForge.net]

Patches item #888380, was opened at 2004-01-31 19:44
Message generated for change (Comment added) made by akuchling
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: A.M. Kuchling (akuchling)
Date: 2004-08-07 17:39

Message:
Logged In: YES 
user_id=11375

Just a reminder -- could you please update this patch so that it can be 
applied?


----------------------------------------------------------------------

Comment By: Jeff Epler (jepler)
Date: 2004-07-10 09: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