Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

<timeout> configuration parameter #128

Closed
mkarg opened this issue Apr 15, 2019 · 2 comments
Closed

<timeout> configuration parameter #128

mkarg opened this issue Apr 15, 2019 · 2 comments
Milestone

Comments

@mkarg
Copy link
Contributor

mkarg commented Apr 15, 2019

Sometimes it happens that the execution of an external task is stuck infinitely, e. g. when a platform specific executable runs into an unforeseen condition where it asks for user input, but the user of the maven exec plugin thought it won't ask anything at any time.

To prevent this, it would be good it a <timeout> parameter (default: inifinite) could be given which will fail the build once the executable needs longer than the given timeout.

@mkarg
Copy link
Contributor Author

mkarg commented Apr 15, 2019

@rfscholte If you think this is a good idea, I would be happy to provide a PR. :-)

mkarg added a commit to mkarg/exec-maven-plugin that referenced this issue Apr 15, 2019
Fixes mojohaus#128.

A new `<timeout>` configuration parameter (default: 0, which means
"none") allows to explicitly terminate a process running for longer that the given time in
minutes, and will fail the build.

Signed-off-by: Markus KARG <[email protected]>
mkarg added a commit to mkarg/exec-maven-plugin that referenced this issue Apr 16, 2019
Fixes mojohaus#128.

A new `<timeout>` configuration parameter (default: 0, which means
"none") allows to explicitly terminate a process running for longer that the given time in
minutes, and will fail the build.

Signed-off-by: Markus KARG <[email protected]>
@mkarg
Copy link
Contributor Author

mkarg commented Jul 27, 2019

🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants