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 the remainder of POSIX signals (plus WINCH and IO) #11

Merged
merged 1 commit into from
Oct 16, 2016
Merged

Add the remainder of POSIX signals (plus WINCH and IO) #11

merged 1 commit into from
Oct 16, 2016

Conversation

bnewbold
Copy link
Contributor

@bnewbold bnewbold commented Oct 8, 2016

I just now noticed that this slightly conflicts with @l0kod's #10; they should merge pretty smoothly though.

Tests pass on my Linux box.
I made these patches in my own time and submit them under the existing LICENSE terms for this repo.


I did a quick check against OS X and OpenBSD. SIGPOLL is supposed to be part of
POSIX.1-2008, with the note that it might be removed in the future, and isn't
in either of those OSes, so removed that. SIGWINCH and SIGIO seem pretty
popular, so did include those (they are in the rust libc crate).

Refs:
http:https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/signal.h.html
http:https://man.openbsd.org/sigaction.2
https://developer.apple.com/legacy/library/documentation/Darwin/Reference/ManPages/man3/signal.3.html

I did a quick check against OS X and OpenBSD. SIGPOLL is supposed to be part of
POSIX.1-2008, with the note that it might be removed in the future, and isn't
in either of those OSes, so removed that. SIGWINCH and SIGIO seem pretty
popular, so did include those (they are in the rust libc crate).

Refs:
http:https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/signal.h.html
http:https://man.openbsd.org/sigaction.2
https://developer.apple.com/legacy/library/documentation/Darwin/Reference/ManPages/man3/signal.3.html
@BurntSushi BurntSushi merged commit 50972e4 into BurntSushi:master Oct 16, 2016
@BurntSushi
Copy link
Owner

Thanks!

This was referenced Oct 16, 2016
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