You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by xi...@apache.org on 2013/01/30 07:34:30 UTC

svn commit: r1440288 - /geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/system/util/ShellExecutor.groovy

Author: xiaming
Date: Wed Jan 30 06:34:29 2013
New Revision: 1440288

URL: http://svn.apache.org/viewvc?rev=1440288&view=rev
Log:
r1439250 cannot fix the stackoverflow issue when running with java 7, so another idea from Jarek to try

Modified:
    geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/system/util/ShellExecutor.groovy

Modified: geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/system/util/ShellExecutor.groovy
URL: http://svn.apache.org/viewvc/geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/system/util/ShellExecutor.groovy?rev=1440288&r1=1440287&r2=1440288&view=diff
==============================================================================
--- geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/system/util/ShellExecutor.groovy (original)
+++ geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/system/util/ShellExecutor.groovy Wed Jan 30 06:34:29 2013
@@ -56,7 +56,7 @@ class ShellExecutor extends LogSupport
         }
     }
     
-    def execute(List args) {
+    def doExecute(List args) {
         def scriptFile = File.createTempFile('shell-executor', '.sh')
         scriptFile.deleteOnExit()
         
@@ -103,17 +103,21 @@ class ShellExecutor extends LogSupport
         }
     }
     
+    def execute(List args) {
+        doExecute(args)
+    }
+    
     def execute() {
         return execute(null)
     }
     
     static def execute(String script, List args) {
-        return new ShellExecutor(script).execute(args)
+        return new ShellExecutor(script).doExecute(args)
     }
     
     static def execute(String script) {
         //return execute(script, null)
-        return new ShellExecutor(script).execute(null) 
+        return new ShellExecutor(script).doExecute(null) 
     }
 }