-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
#2435 Restore signal disposition to previous one #4216
#2435 Restore signal disposition to previous one #4216
Conversation
@bnoordhuis The PR fixes an issue #2435. |
…o be restored in future
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
@vtjnash you want to take another look or should I go ahead and merge?
There is a comment in one of the libuv test files that |
That comment is about https://bugs.python.org/issue20584. Easiest workaround is to default to 128. There's no OS with more signals than that. |
Using 128 hardcoded sounds good too. Just so long as we aren't using a value that is apparently known to be wrong here 😬 |
@bnoordhuis @vtjnash I'v pushed updates, now UV__NSIG constant equals to 128 is used instead of NSIG. |
@bnoordhuis @vtjnash Is the PR ready to merge? |
Merged, thanks! |
This reverts commit b9421d7.
This reverts commit b9421d7. Refs: libuv#4299 Refs: libuv#4248
Hi everyone! It is my first PR :) Glad to have the opportunity to contribute.