Ned Jackson Lovely added the comment:

Hi Luke,

I've prepared two versions of this test. The first, issue5051-take2.diff, 
retains the environ.clear(), but saves and sets COMSPEC and PATH in the same 
update call as the "HELLO" variable.

The second, and in my opinion more reasonable test, makes sure that "HELLO" 
isn't already set, sets it using update, then makes sure it is set properly.

In both cases, the currently running python executable, fetched via 
sys.executable and run using os.popen, is used to print the value, instead of 
the shell's echo. This moves things closer towards cross-platform niceness, and 
removes the dependency on /bin/sh.

Unfortunately, I don't have a Windows machine readily available to test this 
on. Could apply your preferred patch, run it for me, and let me know if you 
have any problems?

Regards,

Ned

----------
Added file: http://bugs.python.org/file29481/issue5051-take3.diff

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue5051>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to