Skip to content

Document most common signals #19245

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
Merged
merged 3 commits into from
Mar 31, 2020
Merged

Document most common signals #19245

merged 3 commits into from
Mar 31, 2020

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Mar 31, 2020 •
edited
Loading

Document individual signals (only the most common signals):
description, default action, availability.

@vstinner
Document most common signals
c89095d 
Document individual signals (only the most common signals):
description, default action, availability.
@vstinner
Copy link
Member Author

@gpshead, @serhiy-storchaka, @pablogsal, @pitrou, @methane: Would you mind to review this doc change?

Does it sound a good idea to you to document the "default action"? The exact behavior might depend on the platform, but IMO it's better to have a little bit inaccurate documentation than no documentation at all.

I can create an issue if someone considers that it's needed (I added skip issue label).

@pablogsal
Copy link
Member

pablogsal commented Mar 31, 2020 •
edited
Loading

I am uneasy documenting the "default" action as you mentioned it can be platform-dependent and is easy to diverge in the future. Also, these default actions are not linked to the signal themselves, is just that the kernel default action on unregistered signals is to kill the process. There is no semantic meaning for instance between SIGUSR1 and killing the process.

but IMO it's better to have a little bit inaccurate documentation than no documentation at all.

I think is better to not say anything than to say something incorrect ;)

@vstinner
Undocument default action
be2bb79 
Only SIGINT and SIGPIPE have portable default action.
@vstinner
Copy link
Member Author

I am uneasy documenting the "default" action as as you mentioned it can be platform dependent and is easy to diverge in the future.

Ok, I remove the default action, except for SIGINT and SIGPIPE: Python has a portable behavior for these two signals, since we install our own signal handler.

@vstinner
Copy link
Member Author

Is it ok to backport such documentation enhancement to Python 3.7 and 3.8?

@pablogsal
Copy link
Member

Is it ok to backport such documentation enhancement to Python 3.7 and 3.8?

I am +1 with the backport

@pablogsal
Copy link
Member

Ok, I remove the default action, except for SIGINT and SIGPIPE: Python has a portable behavior for these two signals, since we install our own signal handler.

👌

pablogsal
pablogsal reviewed Mar 31, 2020
View reviewed changes
Doc/library/signal.rst

Kill signal.

It cannot be caught, blocked, or ignored.
Copy link
Member

@pablogsal pablogsal Mar 31, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

<pedantic> It will be "ignored" if the process is in uninterruptible sleep (under some definition of "ignored")</pedantic>

pablogsal
pablogsal approved these changes Mar 31, 2020
View reviewed changes
Copy link
Member

@pablogsal pablogsal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Thanks for this useful addition! 🎉

@vstinner vstinner merged commit 400e1db into python:master Mar 31, 2020
@vstinner vstinner deleted the signals_doc branch March 31, 2020 17:13
@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Sorry, @vstinner, I could not cleanly backport this to 3.8 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 400e1dbcad93061f1f7ab4735202daaa5e731507 3.8

@bedevere-bot
Copy link

GH-19257 is a backport of this pull request to the 3.8 branch.

vstinner added a commit that referenced this pull request Mar 31, 2020
@vstinner
Document most common signals (GH-19245) (GH-19257)
40e1b04 
Document individual signals (only the most common signals):
description, default action, availability.

(cherry picked from commit 400e1db)
vstinner added a commit that referenced this pull request Mar 31, 2020
@vstinner
Document most common signals (GH-19245) (GH-19258)
ad8e56d 
Document individual signals (only the most common signals):
description, default action, availability.

(cherry picked from commit 400e1db)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pablogsal pablogsal pablogsal approved these changes

Assignees

@vstinner vstinner

Labels
skip issue skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@vstinner @pablogsal @miss-islington @bedevere-bot @the-knights-who-say-ni