Issue 28795: Misleading stating, that SIGINT handler is installed by default
Created on 2016-11-24 22:25 by Jan Velecký, last changed 2022-04-11 14:58 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| issue28795-tip.diff | mdk, 2016-11-29 21:17 | review | ||
| issue28795-2.7.diff | mdk, 2016-11-29 21:17 | review | ||
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 13121 | merged | mdk, 2019-05-06 17:40 | |
| PR 13158 | merged | miss-islington, 2019-05-07 15:28 | |
| Messages (7) | |||
|---|---|---|---|
| msg281664 - (view) | Author: Jan Velecký (Jan Velecký) | Date: 2016-11-24 22:25 | |
Hello, documentation (https://docs.python.org/2/library/signal.html) states, that Python by default installs SIGINT handler which cause KeyboardInterrupt. This is not true everytime according to implementation. "Python installs a small number of signal handlers by default: SIGPIPE ... and SIGINT is translated into a KeyboardInterrupt exception." It should also mention this "SIGINT is installed only, when default handler is set at startup.". Because user can run python script from non-interative shell as background task and SIGINT handler is not installed, regardless although user does not change Python default behaviour. Related SO: http://stackoverflow.com/questions/40775054/capturing-sigint-using-keyboardinterrupt-exception-works-in-terminal-not-in-scr/40785230?noredirect=1 |
|||
| msg281671 - (view) | Author: Julien Palard (mdk) * ![]() |
Date: 2016-11-25 00:13 | |
Maybe something like: > Python installs a small number of signal handlers by default: SIGPIPE is ignored (so write errors on pipes and sockets can be reported as ordinary Python exceptions) and SIGINT (if parent process has not changed it) is translated into a KeyboardInterrupt exception. All of these can be overridden. |
|||
| msg281728 - (view) | Author: Jan Velecký (Jan Velecký) | Date: 2016-11-25 16:32 | |
That's good. |
|||
| msg282047 - (view) | Author: Julien Palard (mdk) * ![]() |
Date: 2016-11-29 21:17 | |
Proposed as patches but english is not my native language so please review carefully. |
|||
| msg341749 - (view) | Author: Julien Palard (mdk) * ![]() |
Date: 2019-05-07 15:27 | |
New changeset e85ef7a7eacdef2f43e6bf2e67f335100e7ef2da by Julien Palard in branch 'master': bpo-28795: Signal documentation: Fix misleading statement. (GH-13121) https://github.com/python/cpython/commit/e85ef7a7eacdef2f43e6bf2e67f335100e7ef2da |
|||
| msg341751 - (view) | Author: Julien Palard (mdk) * ![]() |
Date: 2019-05-07 15:29 | |
Thanks for reporting Jan! It's finally merged \o/ |
|||
| msg341759 - (view) | Author: miss-islington (miss-islington) | Date: 2019-05-07 15:55 | |
New changeset 721729fca4fab9fd11861844880b3f3780015ae0 by Miss Islington (bot) in branch '3.7': bpo-28795: Signal documentation: Fix misleading statement. (GH-13121) https://github.com/python/cpython/commit/721729fca4fab9fd11861844880b3f3780015ae0 |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:58:40 | admin | set | github: 72981 |
| 2019-05-07 15:55:42 | miss-islington | set | nosy:
+ miss-islington messages: + msg341759 |
| 2019-05-07 15:29:33 | mdk | set | status: open -> closed resolution: fixed messages: + msg341751 stage: backport needed -> resolved |
| 2019-05-07 15:28:21 | mdk | set | stage: patch review -> backport needed |
| 2019-05-07 15:28:00 | miss-islington | set | pull_requests: + pull_request13075 |
| 2019-05-07 15:27:53 | mdk | set | messages: + msg341749 |
| 2019-05-06 17:40:14 | mdk | set | stage: patch review pull_requests: + pull_request13034 |
| 2016-11-29 21:17:55 | mdk | set | files: + issue28795-2.7.diff |
| 2016-11-29 21:17:43 | mdk | set | files:
+ issue28795-tip.diff keywords: + patch messages: + msg282047 |
| 2016-11-25 16:32:40 | Jan Velecký | set | messages: + msg281728 |
| 2016-11-25 00:13:05 | mdk | set | nosy:
+ mdk messages: + msg281671 |
| 2016-11-24 22:25:57 | Jan Velecký | create | |
