[issue6649] idlelib/rpc.py missing exit status on exithook

2013-04-07 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Congratulation with your first CPython commit, Roger! -- nosy: +serhiy.storchaka ___ Python tracker ___ ___

[issue6649] idlelib/rpc.py missing exit status on exithook

2013-03-30 Thread Roger Serwy
Roger Serwy added the comment: Thanks for the patch, Guilherme! -- resolution: -> fixed stage: -> committed/rejected status: open -> closed versions: +Python 3.3, Python 3.4 -Python 2.6, Python 3.1 ___ Python tracker

[issue6649] idlelib/rpc.py missing exit status on exithook

2013-03-30 Thread Roundup Robot
Roundup Robot added the comment: 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

[issue6649] idlelib/rpc.py missing exit status on exithook

2013-03-30 Thread Roger Serwy
Changes by Roger Serwy : -- assignee: -> roger.serwy type: -> behavior ___ Python tracker ___ ___ Python-bugs-list mailing list Unsub

[issue6649] idlelib/rpc.py missing exit status on exithook

2012-04-25 Thread Roger Serwy
Changes by Roger Serwy : -- nosy: +terry.reedy ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pytho

[issue6649] idlelib/rpc.py missing exit status on exithook

2012-04-14 Thread Roger Serwy
Roger Serwy added the comment: 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. -- nosy: +serwy ___ Python tracker

[issue6649] idlelib/rpc.py missing exit status on exithook

2009-08-05 Thread Guilherme Polo
New submission from Guilherme Polo : 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. -- components: IDLE files: missing_exitstatus.diff keywords: patch messages: 913