<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
IMHO it might be better to use variable names like WINE_xxx for tests<br>
involving "enumerations" to avoid needlessly altering the environment.<br>
(not that BAR, FOO or FOOBAR are likely to be already used environment<br>
variables)<br>
<span class="HOEnZb"><font color="#888888"><br>
Frédéric<br></font></span></blockquote><div><br></div><div>Thanks! </div><div><br></div><div>We dont really change the environment long term, as the test spawns a process which has its process modified and runs to completion ie any changes are thrown away. However, I do agree that if there was an important environment variable which controlled the shell (in the bug's case it was BASH_xxx, but could be anything) then it would be wise to leave it in place, so such a change probably isnt a bad idea anyway.  (I'll make this change in try 2, as I need to fix the broken NT4 stuff from patch 1 anyway)</div>
<div><br></div><div>Jason</div></div><br>