[ python-Bugs-1730441 ] os._execvpe raises assignment error in python 3000 svn

[SourceForge.net]

Bugs item #1730441, was opened at 2007-06-03 18:17
Message generated for change (Comment added) made by nnorwitz
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=1730441&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: Python Library
Group: Python 3000
>Status: Closed
>Resolution: Fixed
Priority: 3
Private: No
Submitted By: nifan (nifan)
>Assigned to: Neal Norwitz (nnorwitz)
Summary: os._execvpe raises assignment error in python 3000 svn

Initial Comment:
While converting/playing with our project and Python 3000 we hit upon this what we think is a small error in the os.py source.

We communicate with mercurial (hg command) and hit the following error:

http://qna.nu/paste/cnxx/

Thought it seems that the correct way to call subprocess.Popen is with a list not a string, the error it raises is still an variable referenced before assignment error.

Child Exception: http://qna.nu/paste/e5el/

Seems that with Python3000 and _execvpe, the raise statement ending the function uses 'e' before assignment.

Python 2.4 and Python 2.5 seems to do the correct thing and raise an OSError: [Errno 2] No such file or directory.


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

>Comment By: Neal Norwitz (nnorwitz)
Date: 2007-06-03 23:24

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

Nevermind.  I see in the link you have a test and can reproduce the
problem.

Committed revision 55754.


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

Comment By: Neal Norwitz (nnorwitz)
Date: 2007-06-03 22:59

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

Thanks for the report.  Can you test the attached patch?  The semantics
for try/except have changed and this code doesn't look like it was updated.
File Added: os-execvpe.patch

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

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