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

make auto-restart restart the sub-process if it terminates #896


Copy link

@taleinat taleinat commented Jun 5, 2022

Currently, when running watchmedo auto-restart ..., if the sub-process terminates, it is left as a zombie process and watchmedo continues running and waiting for filesystem events.

With this PR, the sub-process will be monitored for termination, and restarted if it terminates.

Alternatively, we could make this exit upon sub-process termination, possibly with a flag to choose between the two behaviours.

This is related to issue #405.

Copy link

BoboTiG commented Jun 5, 2022

Thanks a lot @taleinat !

A small line in the changelog, and we will be good to go :)

Copy link
Contributor Author

taleinat commented Jun 5, 2022

A small line in the changelog, and we will be good to go :)


@BoboTiG BoboTiG merged commit fa806f1 into gorakhargosh:master Jun 6, 2022
13 of 19 checks passed
@taleinat taleinat deleted the auto-restart-handle-subprocess-termination branch June 6, 2022 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

None yet

2 participants