Re: svn commit: r948853 - /ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Execute.java

2010-05-28 Thread Jesse Glick
On 05/28/2010 11:31 AM, Stefan Bodewig wrote: I'm thinking about putting together a tiny jar that contains a command line Java main class that compares System.getEnv and Ant's traditional way on the current system and then ask on our user list whether anybody there was willing to run it on the mo

Re: svn commit: r948853 - /ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Execute.java

2010-05-28 Thread Bruce Atherton
On 28/05/2010 8:31 AM, Stefan Bodewig wrote: I'm thinking about putting together a tiny jar that contains a command line Java main class that compares System.getEnv and Ant's traditional way on the current system and then ask on our user list whether anybody there was willing to run it on the mor

Re: svn commit: r948853 - /ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Execute.java

2010-05-28 Thread Matt Benson
On May 28, 2010, at 10:31 AM, Stefan Bodewig wrote: On 2010-05-28, Jesse Glick wrote: On 05/27/2010 11:27 AM, Stefan Bodewig wrote: Use System.getenv where available. Are we sure it does the same thing as our current code? We have some hacks for "exotic" platforms like encoding tweaks

Re: svn commit: r948853 - /ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Execute.java

2010-05-28 Thread Stefan Bodewig
On 2010-05-28, Jesse Glick wrote: > On 05/27/2010 11:27 AM, Stefan Bodewig wrote: >>> Use System.getenv where available. >> Are we sure it does the same thing as our current code? >> We have some hacks for "exotic" platforms like encoding tweaks or >> support for logicals on OpenVMS - your patch

Re: svn commit: r948853 - /ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Execute.java

2010-05-28 Thread Jesse Glick
On 05/27/2010 11:27 AM, Stefan Bodewig wrote: Use System.getenv where available. Are we sure it does the same thing as our current code? We have some hacks for "exotic" platforms like encoding tweaks or support for logicals on OpenVMS - your patch will skip the addVMSLogicals invocation, for e