After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 703169 - CTRL+C is not working with Sudoku started from terminal
CTRL+C is not working with Sudoku started from terminal
Status: RESOLVED FIXED
Product: gnome-sudoku
Classification: Applications
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: gnome-sudoku-maint
gnome-sudoku-maint
Depends on:
Blocks:
 
 
Reported: 2013-06-27 08:30 UTC by celine.LEGUAY
Modified: 2013-11-08 03:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Properly die on SIGINT (947 bytes, patch)
2013-11-08 03:57 UTC, Michael Catanzaro
committed Details | Review

Description celine.LEGUAY 2013-06-27 08:30:44 UTC
1.Bot
2.Open a terminal
3.Type 'gnome-sudoku'
4.Press "CTRL + C"
Expected result:
Sudoku game should stop
Actual result:
Nothing happens
Comment 1 alexandru.gheorghita88 2013-07-26 09:31:45 UTC
You can only close gnome-sudoku with CTRL+C if you are root.
Comment 2 Michael Catanzaro 2013-11-08 03:55:56 UTC
It's because Python catches SIGINT and translates it into a Python exception named KeyboardInterrupt, and somewhere we are ignoring all exceptions. I took a quick look but didn't find where. Why being root affects this is quite a mystery.

Quick hack incoming because the Python version is (admittedly very slowly) approaching the end of its life; anyone who finds the real problem should feel free to revert it.
Comment 3 Michael Catanzaro 2013-11-08 03:57:28 UTC
The following fix has been pushed:
07a8392 Properly die on SIGINT
Comment 4 Michael Catanzaro 2013-11-08 03:57:30 UTC
Created attachment 259234 [details] [review]
Properly die on SIGINT