-
Notifications
You must be signed in to change notification settings - Fork 1k
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 SIGRTMAX and SIGRTMIN on linux-like systems #2304
Conversation
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @JohnTitor (or someone else) soon. Please see the contribution instructions for more information. |
Thanks! |
📌 Commit 91dd411 has been approved by |
Add SIGRTMAX and SIGRTMIN on linux-like systems Fixes #1883. This is based on #2195, which has not seen activity in a while. The original issue was resolved by making `__libc_current_sigrtmax` and `__libc_current_sigrtmax` `pub`. I also added a test similar to `errqueue`'s, where the actual value of the macro is compared to `libc`'s implementation. Closes #2195
Looks like the Should I remove the |
💔 Test failed - checks-actions |
Ah yes! I forgot that |
Done. 👍 |
@bors r+ |
📌 Commit 8af1a48 has been approved by |
Add SIGRTMAX and SIGRTMIN on linux-like systems Fixes #1883. This is based on #2195, which has not seen activity in a while. The original issue was resolved by making `__libc_current_sigrtmax` and `__libc_current_sigrtmax` `pub`. I also added a test similar to `errqueue`'s, where the actual value of the macro is compared to `libc`'s implementation. Closes #2195
💔 Test failed - checks-actions |
@bors retry |
☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13 |
Fixes #1883. This is based on #2195, which has not seen activity in a while.
The original issue was resolved by making
__libc_current_sigrtmax
and__libc_current_sigrtmax
pub
.I also added a test similar to
errqueue
's, where the actual value of the macro is compared tolibc
's implementation.Closes #2195