Remove redundant runSimpleBashScript code. It can call itself with a timeout set to 0 since the constructor will set that to the default timeout.
Project: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/commit/2b56edbd Tree: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/tree/2b56edbd Diff: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/diff/2b56edbd Branch: refs/heads/qemu-img Commit: 2b56edbd1a76082ff2158e708414d2ceb370fc2f Parents: dba0df5 Author: Wido den Hollander <[email protected]> Authored: Wed Feb 6 16:29:03 2013 +0100 Committer: Wido den Hollander <[email protected]> Committed: Thu Feb 14 15:01:58 2013 +0100 ---------------------------------------------------------------------- utils/src/com/cloud/utils/script/Script.java | 15 +-------------- 1 files changed, 1 insertions(+), 14 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/2b56edbd/utils/src/com/cloud/utils/script/Script.java ---------------------------------------------------------------------- diff --git a/utils/src/com/cloud/utils/script/Script.java b/utils/src/com/cloud/utils/script/Script.java index 9731ad1..32e4a6c 100755 --- a/utils/src/com/cloud/utils/script/Script.java +++ b/utils/src/com/cloud/utils/script/Script.java @@ -454,20 +454,7 @@ public class Script implements Callable<String> { } public static String runSimpleBashScript(String command) { - - Script s = new Script("/bin/bash"); - s.add("-c"); - s.add(command); - - OutputInterpreter.OneLineParser parser = new OutputInterpreter.OneLineParser(); - if (s.execute(parser) != null) - return null; - - String result = parser.getLine(); - if (result == null || result.trim().isEmpty()) - return null; - else - return result.trim(); + return Script.runSimpleBashScript(command, 0); } public static String runSimpleBashScript(String command, int timeout) {
