Skip to content
This repository has been archived by the owner on Apr 16, 2024. It is now read-only.

Enhance the forked tomcat shutdown mojo to ensure it waits for tomcat… #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lucasgut
Copy link

… to shutdown gracefully. - The start and stop mojos now use a temporary file to indicate if an instance is running. - Add a new configuration property tomcatShutdownTimeout to indicate how many seconds the stop mojo should wait for tomcat to shutdown before timing out.

… to shutdown gracefully. - The start and stop mojos now use a temporary file to indicate if an instance is running. - Add a new configuration property tomcatShutdownTimeout to indicate how many seconds the stop mojo should wait for tomcat to shutdown before timing out.
@lucasgut
Copy link
Author

I find the forked mode useful for running jacoco code coverage reports. This change should ensure that the forked stop mojo waits for tomcat to finish otherwise the jacoco reports stage would run before the coverage results are written to disk.

@jbellmann
Copy link
Member

@lucasgut

Are you still interested in this PR merged? Sorry for the delay. But I do not use Tomcat anymore for a while now and did not see your PR.

Regards
Jörg

@jbellmann jbellmann closed this Aug 29, 2016
@jbellmann jbellmann reopened this Aug 29, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants