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)
}
}