Issue 6649: idlelib/rpc.py missing exit status on exithook (original) (raw)
This issue has been migrated to GitHub: https://github.com/python/cpython/issues/50898
classification
Title: | idlelib/rpc.py missing exit status on exithook | ||
---|---|---|---|
Type: | behavior | Stage: | resolved |
Components: | IDLE | Versions: | Python 3.3, Python 3.4, Python 2.7 |
process
Status: | closed | Resolution: | fixed |
---|---|---|---|
Dependencies: | Superseder: | ||
Assigned To: | roger.serwy | Nosy List: | gpolo, python-dev, roger.serwy, serhiy.storchaka, terry.reedy |
Priority: | low | Keywords: | patch |
Created on 2009-08-05 16:21 by gpolo, last changed 2022-04-11 14:56 by admin. This issue is now closed.
Files | |||
---|---|---|---|
File name | Uploaded | Description | Edit |
missing_exitstatus.diff | gpolo,2009-08-05 16:21 | review |
Messages (5) | ||
---|---|---|
msg91322 - (view) | Author: Guilherme Polo (gpolo) * ![]() |
Date: 2009-08-05 16:21 |
SocketIO.exithook on idlelib/rpc.py is missing the exit status, this is a minor issue since both client and server used on IDLE override this method to do something else. | ||
msg158295 - (view) | Author: Roger Serwy (roger.serwy) * ![]() |
Date: 2012-04-14 22:47 |
The existing code will raise an error since os._exit requires an argument. http://docs.python.org/library/os.html#os._exit The patch looks good to me. | ||
msg185616 - (view) | Author: Roundup Robot (python-dev) ![]() |
Date: 2013-03-31 06:18 |
New changeset e044d22d2f61 by Roger Serwy in branch '2.7': #6649: Fixed missing exit status in IDLE. Patch by Guilherme Polo. http://hg.python.org/cpython/rev/e044d22d2f61 New changeset cd2c613527ed by Roger Serwy in branch '3.3': #6649: Fixed missing exit status in IDLE. Patch by Guilherme Polo. http://hg.python.org/cpython/rev/cd2c613527ed New changeset abbbe1f90635 by Roger Serwy in branch 'default': #6649: merge with 3.3. http://hg.python.org/cpython/rev/abbbe1f90635 | ||
msg185617 - (view) | Author: Roger Serwy (roger.serwy) * ![]() |
Date: 2013-03-31 06:20 |
Thanks for the patch, Guilherme! | ||
msg186180 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * ![]() |
Date: 2013-04-07 07:23 |
Congratulation with your first CPython commit, Roger! |
History | |||
---|---|---|---|
Date | User | Action | Args |
2022-04-11 14:56:51 | admin | set | github: 50898 |
2013-04-07 07:23:22 | serhiy.storchaka | set | nosy: + serhiy.storchakamessages: + |
2013-03-31 06:20:57 | roger.serwy | set | status: open -> closedversions: + Python 3.3, Python 3.4, - Python 2.6, Python 3.1messages: + resolution: fixedstage: resolved |
2013-03-31 06🔞30 | python-dev | set | nosy: + python-devmessages: + |
2013-03-31 04:07:07 | roger.serwy | set | assignee: roger.serwytype: behavior |
2012-04-26 00:28:50 | roger.serwy | set | nosy: + terry.reedy |
2012-04-14 22:47:30 | roger.serwy | set | nosy: + roger.serwymessages: + |
2009-08-05 16:21:07 | gpolo | create |