Execute a external program within a groovy script and capture the output

Question

I need to write a groovy script, that is executing a external program and print the output of that program to the console.

Here is the regarding code snippet:

def pmdCommand = "${scriptDir}/run.sh pmd -d ${filesToAnalyse}"

def sout = new StringBuffer()
def serr = new StringBuffer()

def process = pmdCommand.execute()
process.consumeProcessOutput(sout, serr)
process.waitFor()
if (process.exitValue() !=0 ) {
    System.err << serr.toString()
    System.exit(-1)
} 
else {
    System.out << sout.toString()
    System.exit(0)
}

I did something similar in Java, but I can't translate it to groovy.

StringBuffer output = new StringBuffer();
String s = null;

try {
    Process p = Runtime.getRuntime().exec(command);
    p.waitFor();

    BufferedReader stdInput = new BufferedReader(new InputStreamReader(p.getInputStream()));
    BufferedReader stdError = new BufferedReader(new InputStreamReader(p.getErrorStream()));

    while ((s = stdInput.readLine()) != null) {
        System.out.println(s);
    }

    while ((s = stdError.readLine()) != null) {
        System.err.println(s);
    }

    System.exit(0);
}
catch (Exception e) {
    e.printStackTrace();
    System.exit(-1);
}

Update: I seems the waitFor() never returns and blocks the execution

Solution provided by Emmanuel Rosa:

def pmdCommand = "/usr/src/app/lib/pmd/bin/run.sh pmd -d ${filesToAnalyse} -f codeclimate -R ${ruleset} -l apex -v 35"

def sout = new StringBuffer()
def serr = new StringBuffer()

def process = pmdCommand.execute()
process.consumeProcessOutput(sout, serr)
process.waitForProcessOutput()

System.out << sout.toString()
System.exit(0)

@tim_yates I updated my groovy code above and It seems the waitFor() method never returns — David Renz
– David Renz, Commented May 11, 2016 at 15:02

Emmanuel Rosa · Accepted Answer · 2016-05-11 16:08:34Z

6

The documentation states that consumeProcessOutput()...

Gets the output and error streams from a process and reads them to keep the process from blocking due to a full output buffer. The processed stream data is appended to the supplied OutputStream. For this, two Threads are started, so this method will return immediately.

So far so good. Here's the important part...

The threads will not be join()ed, even if waitFor() is called.

And the solution...

To wait for the output to be fully consumed call waitForProcessOutput().

So what you can do is replace process.waitFor() with process.waitForProcessOutput().

answered May 11, 2016 at 16:08

Emmanuel Rosa

9,8752 gold badges17 silver badges21 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

David Renz Over a year ago

Thank you very much, this was not the only problem, but definitely a part of the issue.

Collectives™ on Stack Overflow

Execute a external program within a groovy script and capture the output

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related