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 697344 - Ctrl+C doesn't close gnome-tweak-tool
Ctrl+C doesn't close gnome-tweak-tool
Status: RESOLVED FIXED
Product: gnome-tweak-tool
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME Tweak Tool maintainer(s)
GNOME Tweak Tool maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2013-04-05 12:44 UTC by Guillaume Desmottes
Modified: 2017-06-11 21:17 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Ctrl-C signal (817 bytes, patch)
2013-12-15 04:20 UTC, Alex Muñoz
committed Details | Review

Description Guillaume Desmottes 2013-04-05 12:44:42 UTC
- Start gnome-tweak-tool from a terminal
- In this terminal hits Ctrl+C
- Nothing happens
- Try closing gnome-tweak-tool's window. We get this error but the window isn't closed

Traceback (most recent call last):
  • File "/usr/lib64/python2.7/site-packages/gi/overrides/__init__.py", line 56 in wrapper
    def wrapper(*args, **kwargs): KeyboardInterrupt

Comment 1 Michael Catanzaro 2013-12-09 20:52:29 UTC
Easiest fix would be to reset the default signal handler, like in https://git.gnome.org/browse/gnome-sudoku/commit/?id=f5a382d480e33ed98e81b0f2888b7c2d4505360d  (the deletion there is a typo)
Comment 2 Alex Muñoz 2013-12-15 04:20:52 UTC
Created attachment 264212 [details] [review]
Ctrl-C signal
Comment 3 John Stowers 2014-01-12 14:04:18 UTC
Does this still exist with recent pygobject?
Comment 4 Alex Muñoz 2014-01-12 21:05:59 UTC
yes, I tested it using the last version.
also in this bug report was discussed the issue.

https://bugzilla.gnome.org/show_bug.cgi?id=622084
Comment 5 Jeremy Bicha 2017-06-06 23:58:53 UTC
Review of attachment 264212 [details] [review]:

Thank you! Sorry about the delay.
Comment 6 Michael Catanzaro 2017-06-07 03:12:37 UTC
Imports should probably be alphabetized!
Comment 7 Jeremy Bicha 2017-06-11 21:17:48 UTC
Alex, I wasn't sure if you were still active in GNOME and this patch is really useful for development so I went ahead and pushed it.

Thank you!