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

Feature request: handle SIGHUP #26

Open
65278 opened this issue Apr 26, 2018 · 0 comments
Open

Feature request: handle SIGHUP #26

65278 opened this issue Apr 26, 2018 · 0 comments

Comments

@65278
Copy link
Contributor

65278 commented Apr 26, 2018

We need to extend the signal handler to SIGHUP.
The rationale is: schaufel runs as a child of listen.pl which runs as a child of tmux.
In this setup, killing listen.pl to stop schaufel leads to data loss.

I.e. tmux receives SIGCHLD and sends a SIGHUP to schaufel. Schaufel only handles SIGINT/SIGTERM. As the internal state handler is skipped, data is not flushed.
65278 pushed a commit that referenced this issue May 3, 2018
src/main.c: handle sighup
3052d4c 
we don't really want to lose data
just because the terminal has hung up

Ref: #26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@65278