Support ignoreExitValue on NodeTask

[konrad-garus]

ExecSpec in core Gradle supports the ignoreExitValue flag. When set, it ignores exit value of the process (otherwise an exception is thrown). Please expose this flag in NodeTask as well.

Optionally, also add a way to customize the ExecSpec directly. For instance accept a closure to run on the spec directly, like this in ExecRunner:

return this.project.exec( {
            it.executable = realExec
            it.args = realArgs

            if ( this.environment != null )
            {
                it.environment = this.environment
            }

            if ( this.workingDir != null )
            {
                it.workingDir = this.workingDir
            }
            // execOverrides is the new closure:
            execOverrides(it)
        } )

Why is it needed? I use it to run integration tests, and I want to shut down my server after running the tests. The tests may fail, but I don't want the build to stop.

[konrad-garus]

If you're interested I can create a PR for it.

[srs]

Yes. I welcome a pull request :-)

// srs

Den 10. feb. 2014 kl. 11:11 skrev Konrad Garus notifications@github.com:

If you're interested I can create a PR for it.

—
Reply to this email directly or view it on GitHub.

[konrad-garus]

With the closure as well, or just the direct ignoreExitValue property? It's your call on the API.

[srs]

A closure would be more flexible so let's try that :-)

[konrad-garus]

It's actually a fix on the Node plugin: srs/gradle-node-plugin#6