You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by km...@apache.org on 2011/07/14 02:40:00 UTC

svn commit: r1146529 - /openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/RemoteServer.java

Author: kmalhi
Date: Thu Jul 14 00:39:59 2011
New Revision: 1146529

URL: http://svn.apache.org/viewvc?rev=1146529&view=rev
Log:
Fixed path separator to be independent of underlying OS. Was set in a unix compatible way so would break on windows. 

Modified:
    openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/RemoteServer.java

Modified: openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/RemoteServer.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/RemoteServer.java?rev=1146529&r1=1146528&r2=1146529&view=diff
==============================================================================
--- openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/RemoteServer.java (original)
+++ openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/RemoteServer.java Thu Jul 14 00:39:59 2011
@@ -183,12 +183,12 @@ public class RemoteServer {
                     argsList.add("-Dcatalina.ext.dirs=" + tlib.getAbsolutePath());
                     argsList.add("-Dopenejb.servicemanager.enabled=" + Boolean.getBoolean("openejb.servicemanager.enabled"));
                     argsList.add("-classpath");
-
+                    String ps = File.pathSeparator;
                     if (commonsLoggingJar.exists()) {
-                        argsList.add(bootstrapJar.getAbsolutePath() + ":" + juliJar.getAbsolutePath() + ":" + commonsLoggingJar.getAbsolutePath());
+                        argsList.add(bootstrapJar.getAbsolutePath() + ps + juliJar.getAbsolutePath() + ps + commonsLoggingJar.getAbsolutePath());
 
                     } else {
-                        argsList.add(bootstrapJar.getAbsolutePath() + ":" + juliJar.getAbsolutePath());
+                        argsList.add(bootstrapJar.getAbsolutePath() + ps + juliJar.getAbsolutePath());
                     }
 
                     argsList.add("org.apache.catalina.startup.Bootstrap");
@@ -277,7 +277,6 @@ public class RemoteServer {
             out.close();
             if (verbose) System.out.println("[] CONNECTED IN " + (this.tries - tries));
         } catch (Exception e) {
-            //System.out.println(e.getMessage());
             if (tries < 2) {
                 if (verbose) System.out.println("[] CONNECT ATTEMPTS FAILED ( " + (this.tries - tries) + " tries)");
                 return false;