[openejb-scm] [2505] trunk/openejb1/modules/itests/src/java/org/openejb/test/RemoteTestServer.java: Fixed test server startup code for when spaces are in path

[dblevins]

Title: [2505] trunk/openejb1/modules/itests/src/java/org/openejb/test/RemoteTestServer.java: Fixed test server startup code for when spaces are in path

Revision

2505

Author

dblevins

Date

2006-02-25 00:31:23 -0500 (Sat, 25 Feb 2006)

Log Message

Fixed test server startup code for when spaces are in path

Modified Paths

• trunk/openejb1/modules/itests/src/java/org/openejb/test/RemoteTestServer.java

Diff

Modified: trunk/openejb1/modules/itests/src/java/org/openejb/test/RemoteTestServer.java (2504 => 2505)

--- trunk/openejb1/modules/itests/src/java/org/openejb/test/RemoteTestServer.java	2006-02-25 05:03:38 UTC (rev 2504)
+++ trunk/openejb1/modules/itests/src/java/org/openejb/test/RemoteTestServer.java	2006-02-25 05:31:23 UTC (rev 2505)
@@ -108,7 +108,10 @@
                 File lib = home.getDirectory("lib");
                 File openejbJar = new File(lib, "openejb-core-" + version + ".jar");
 
-                Process server = Runtime.getRuntime().exec("java -jar "+openejbJar.getAbsolutePath()+" start -nowait");
+                //DMB: If you don't use an array, you get problems with jar paths containing spaces
+                // the command won't parse correctly
+                String[] args = {"java", "-jar", openejbJar.getAbsolutePath(), "start"};
+                Process server = Runtime.getRuntime().exec(args);
 
                 // Pipe the processes STDOUT to ours
                 InputStream out = server.getInputStream();
@@ -117,8 +120,6 @@
                 serverOut.setDaemon(true);
                 serverOut.start();
 
-
-
                 // Pipe the processes STDERR to ours
                 InputStream err = server.getErrorStream();
                 Thread serverErr = new Thread(new Pipe(err, System.err));