[Patches] [ python-Patches-1609282 ] #1603424 subprocess.py wrongly claims 2.2 compatibility.

[SourceForge.net]

Patches item #1609282, was opened at 2006-12-05 16:16
Message generated for change (Comment added) made by astrand
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1609282&group_id=5470

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
>Resolution: Rejected
Priority: 5
Private: No
Submitted By: Robert Carr (racarr)
>Assigned to: Peter Åstrand (astrand)
Summary: #1603424 subprocess.py wrongly claims 2.2 compatibility.

Initial Comment:
Simple fix restoring 2.2 compatibility in subprocess.py. This makes more sense than a list comprehension or constructing sets in my opinion even ignoring the bug. 

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

>Comment By: Peter Åstrand (astrand)
Date: 2007-01-07 10:03

Message:
Logged In: YES 
user_id=344921
Originator: NO

This patch is rejected, due to the problem described in gbrandl:s comment.
Another fix has been submitted, though, which solves bug #1603424. 

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

Comment By: Georg Brandl (gbrandl)
Date: 2006-12-08 21:55

Message:
Logged In: YES 
user_id=849994
Originator: NO

This patch changes semantics: if two names refer to the same fd, it is
attempted to be closed twice.

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

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1609282&group_id=5470