ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stefan Bodewig <bode...@apache.org>
Subject Re: [patch] new file: jakarta-ant/src/script/antRun.pl
Date Wed, 14 Nov 2001 09:41:57 GMT
Hi Jeff,

I noticed you've commented out the chdir call in the script - is that
intended?

You seem to prefix the directory to the command to execute, this is
not what antRun does - there is a difference between

cd /tmp
ls

and 

/tmp/ls

at least in most situations 8-)

I guess I'm simply not understanding the script, which may very well
be as my Perl has become more than a little rusty.

Also, I just now realized that your first patches applied to Exec.java
- this file isn't used anywhere anymore, <exec> corresponds to
ExecTask.java which in turn uses Execute.java, I guess you want to do
something like the attached patch.

Stefan

Index: src/main/org/apache/tools/ant/taskdefs/Execute.java
===================================================================
RCS file: /home/cvs/jakarta-ant/src/main/org/apache/tools/ant/taskdefs/Execute.java,v
retrieving revision 1.24
diff -u -r1.24 Execute.java
--- src/main/org/apache/tools/ant/taskdefs/Execute.java	2001/11/02 07:56:52	1.24
+++ src/main/org/apache/tools/ant/taskdefs/Execute.java	2001/11/14 09:29:08
@@ -119,6 +119,11 @@
             // OS/2 - use same mechanism as Windows 2000
             shellLauncher = new WinNTCommandLauncher(new CommandLauncher());
         }
+        else if ( Os.isFamily("netware") ) {
+            // Novell Netware - use the Perl script
+            shellLauncher = new ScriptCommandLauncher("bin/antRun.pl", 
+                                                      new CommandLauncher());
+        }
         else if ( Os.isFamily("windows") ) {
             // Windows.  Need to determine which JDK we're running in
 

--
To unsubscribe, e-mail:   <mailto:ant-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:ant-dev-help@jakarta.apache.org>


Mime
View raw message