Duncan,

Would you like a patch already :-) Bootstrap needs to use
File.pathSeparator. Also I had a little fun and wrote Bootboot.java to
save me finding tools.jar. Take it or leave it :-)

Conor



? Bootboot.class
? Bootboot.java
? Bootstrap.class
? patch.txt
? temp
Index: Bootstrap.java
===================================================================
RCS file: /home/cvs/jakarta-ant/proposal/anteater/bootstrap/Bootstrap.java,v
retrieving revision 1.1
diff -u -r1.1 Bootstrap.java
--- Bootstrap.java      2000/12/06 08:08:29     1.1
+++ Bootstrap.java      2000/12/06 10:06:57
@@ -83,7 +83,7 @@
         String[] cmdarray = new String[9];
         cmdarray[0] = "java";
         cmdarray[1] = "-cp";
-        cmdarray[2] = base + "bootstrap/temp/main:" + base + "lib/jaxp.jar:" +
+        cmdarray[2] = base + "bootstrap/temp/main" + File.pathSeparator + base 
+ "lib/jaxp.jar" + File.pathSeparator +
                       base + "lib/crimson.jar";
         cmdarray[3] = "org.apache.ant.cli.Main";
         cmdarray[4] = "-taskpath";
@@ -232,7 +232,10 @@
         // System.out.println("user.dir=" + System.getProperty("user.dir"));
         // System.out.println("Scanning: " + dir);
         String[] files = dir.list();
-        // System.out.println("Files: " + files);
+        if (files == null) {
+            return;
+        }
+        
         for (int i = 0; i < files.length; i++) {
             File f = new File(dir, files[i]);
             if (f.isDirectory()) {
@@ -255,7 +258,7 @@
                                                                  "javac");     
   
         String[] args = new String[sources.size() + 4];
         args[0] = "-classpath";
-        args[1] = base + "bootstrap/temp/main:" + base + "lib/jaxp.jar:" + 
+        args[1] = base + "bootstrap/temp/main" + File.pathSeparator + base + 
"lib/jaxp.jar" + File.pathSeparator + 
                   base + "lib/crimson.jar";
         args[2] = "-d";
         args[3] = dest;

Attachment: Bootboot.java
Description: Binary data

Reply via email to