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.

Sign up for GitHub

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

Jump to bottom

Safer cleanup on SIGTERM #366

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Safer cleanup on SIGTERM #366

wants to merge 6 commits into from

Commits on Apr 19, 2023

  1. Set up cleanup process on init to avoid signal-safety issues

    @lionel-
    lionel- committed Apr 19, 2023
    Configuration menu
    Copy the full SHA
    2a5ca8e View commit details
    Browse the repository at this point in the history

Commits on Apr 25, 2023

  1. Clean up process on quit

    @lionel-
    lionel- committed Apr 25, 2023
    Configuration menu
    Copy the full SHA
    e25c5ff View commit details
    Browse the repository at this point in the history

  2. Ignore SIGTERM in cleanup processes

    @lionel-
    lionel- committed Apr 25, 2023
    Configuration menu
    Copy the full SHA
    1b93d22 View commit details
    Browse the repository at this point in the history

  3. Add test for sigkilled parent process

    @lionel-
    lionel- committed Apr 25, 2023
    Configuration menu
    Copy the full SHA
    a9a55bb View commit details
    Browse the repository at this point in the history

  4. Don't need to exit cleanup process on unload

    @lionel-
    lionel- committed Apr 25, 2023
    Configuration menu
    Copy the full SHA
    5c35551 View commit details
    Browse the repository at this point in the history

  5. Fix -Wunused-result warning

    @lionel-
    lionel- committed Apr 25, 2023
    Configuration menu
    Copy the full SHA
    7b871ed View commit details
    Browse the repository at this point in the history