[jbosstools-commits] JBoss Tools SVN: r31207 - workspace/rstryker/andre2/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/server/internal/v7.

[jbosstools-commits at lists.jboss.org]

Author: rob.stryker at jboss.com
Date: 2011-05-11 03:43:38 -0400 (Wed, 11 May 2011)
New Revision: 31207

Modified:
   workspace/rstryker/andre2/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/server/internal/v7/LocalJBoss7ServerRuntime.java
Log:
Adding quotes to ensure default args for jboss-7 installations with spaces functions properly

Modified: workspace/rstryker/andre2/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/server/internal/v7/LocalJBoss7ServerRuntime.java
===================================================================
--- workspace/rstryker/andre2/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/server/internal/v7/LocalJBoss7ServerRuntime.java	2011-05-11 07:33:10 UTC (rev 31206)
+++ workspace/rstryker/andre2/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/server/internal/v7/LocalJBoss7ServerRuntime.java	2011-05-11 07:43:38 UTC (rev 31207)
@@ -31,9 +31,9 @@
 	@Override
 	public String getDefaultRunArgs() {
 		IPath loc = getRuntime().getLocation();
-		return "-mp " //$NON-NLS-1$
+		return "-mp \"" //$NON-NLS-1$
 				+ loc.append("modules").toString() //$NON-NLS-1$ 
-				+ " -logmodule org.jboss.logmanager -jaxpmodule javax.xml.jaxp-provider org.jboss.as.standalone"; //$NON-NLS-1$
+				+ "\" -logmodule org.jboss.logmanager -jaxpmodule javax.xml.jaxp-provider org.jboss.as.standalone"; //$NON-NLS-1$
 	}
 
 	@Override
@@ -44,8 +44,8 @@
 		return "-server -Xms64m -Xmx512m -XX:MaxPermSize=256m " //$NON-NLS-1$
 				+ "-Dorg.jboss.resolver.warning=true -Dsun.rmi.dgc.client.gcInterval=3600000 " //$NON-NLS-1$
 				+ "-Dsun.rmi.dgc.server.gcInterval=3600000 " //$NON-NLS-1$
-				+ "-Dorg.jboss.boot.log.file=" + bootLog.toString() //$NON-NLS-1$
-				+ " -Dlogging.configuration=file:" + logConfig.toString() //$NON-NLS-1$
-				+ " -Djboss.home.dir=" + loc.toString(); //$NON-NLS-1$"
+				+ "\"-Dorg.jboss.boot.log.file=" + bootLog.toString() //$NON-NLS-1$
+				+ "\" \"-Dlogging.configuration=file:" + logConfig.toString() //$NON-NLS-1$
+				+ "\" \"-Djboss.home.dir=" + loc.toString() + "\""; //$NON-NLS-1$ //$NON-NLS-2$"
 	}
 }