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

Add SIGUSR1 and SIGUSR2 #6

Merged
merged 2 commits into from
Apr 29, 2016
Merged

Add SIGUSR1 and SIGUSR2 #6

merged 2 commits into from
Apr 29, 2016

Conversation

jwilm
Copy link
Contributor

@jwilm jwilm commented Apr 29, 2016

It's often useful to be able to handle SIGUSR1 and SIGUSR2 in an application, and this wasn't yet possible with chan-signal.


This change is Reviewable

test_many failed to receive the second signal when they were sent
simultaneously.
Allows applications to handle SIGUSR1 and SIGUSR2.
@BurntSushi BurntSushi merged commit 884cdad into BurntSushi:master Apr 29, 2016
@BurntSushi
Copy link
Owner

Looks great, thank you! :-)

@BurntSushi
Copy link
Owner

OK, I published chan-signal 0.1.6 which includes this PR. Thanks again!

@jwilm
Copy link
Contributor Author

jwilm commented Apr 29, 2016

Oh cool, thanks! I can remove the git dependency already :)

@jwilm jwilm deleted the sigusr branch April 29, 2016 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants