Author: rmannibucau Date: Tue Apr 29 18:06:07 2014 New Revision: 1591037 URL: http://svn.apache.org/r1591037 Log: TOMEE-1186 supporting system props on ExecRunner
Modified: tomee/tomee/trunk/maven/tomee-maven-plugin/src/main/java/org/apache/openejb/maven/plugin/runner/ExecRunner.java Modified: tomee/tomee/trunk/maven/tomee-maven-plugin/src/main/java/org/apache/openejb/maven/plugin/runner/ExecRunner.java URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/maven/tomee-maven-plugin/src/main/java/org/apache/openejb/maven/plugin/runner/ExecRunner.java?rev=1591037&r1=1591036&r2=1591037&view=diff ============================================================================== --- tomee/tomee/trunk/maven/tomee-maven-plugin/src/main/java/org/apache/openejb/maven/plugin/runner/ExecRunner.java (original) +++ tomee/tomee/trunk/maven/tomee-maven-plugin/src/main/java/org/apache/openejb/maven/plugin/runner/ExecRunner.java Tue Apr 29 18:06:07 2014 @@ -135,9 +135,11 @@ public class ExecRunner { final ProcessBuilder builder = new ProcessBuilder(params.toArray(new String[params.size()])).directory(distribOutput); + final String additionalArgs = System.getProperty("additionalSystemProperties"); + final String existingOpts = System.getenv("CATALINA_OPTS"); final String catalinaOpts = config.getProperty("catalinaOpts"); - if (catalinaOpts != null) { // inherit from existing env - builder.environment().put("CATALINA_OPTS", catalinaOpts); + if (catalinaOpts != null || existingOpts != null || additionalArgs != null) { // inherit from existing env + builder.environment().put("CATALINA_OPTS", identityOrEmpty(catalinaOpts) + " " + identityOrEmpty(existingOpts) + " " + identityOrEmpty(additionalArgs)); } boolean redirectOut = false; @@ -160,6 +162,10 @@ public class ExecRunner { System.out.println("Exited Successfully!"); } + private static String identityOrEmpty(final String value) { + return (value != null ? value : ""); + } + private ExecRunner() { // no-op }