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 326610 - Gnome-Nettool 1.4.1 Tool > Quit menu item does nothing
Gnome-Nettool 1.4.1 Tool > Quit menu item does nothing
Status: RESOLVED FIXED
Product: gnome-nettool
Classification: Applications
Component: general
2.13.x
Other Linux
: Normal minor
: ---
Assigned To: Germán Poo-Caamaño
Rodrigo Moya
Depends on:
Blocks:
 
 
Reported: 2006-01-11 15:06 UTC by Daniel Holbach
Modified: 2006-01-28 18:26 UTC
See Also:
GNOME target: 2.14.x
GNOME version: 2.13/2.14

Attachments
connect the activate signal to gn_quit_app() callback (1.92 KB, patch)
2006-01-28 16:10 UTC, Brent Smith (smitten) 		committed Details | Review

Description Daniel Holbach 2006-01-11 15:06:15 UTC 
Forwarded from: http://bugzilla.ubuntu.com/show_bug.cgi?id=21766

Dapper F#2 Live: Network Tools 1.4.1 "Tool > Quit" menu item doesn't ccause the program to exit (nothing happens.) The <Ctrl><Q> shortcut does not work either. Have to click the [X] box to exit the program
Comment 1 Josep Puigdemont 2006-01-28 14:58:50 UTC 
I can confirm this bug is also present in latest CVS version (2.13.x).
Comment 2 Brent Smith (smitten) 2006-01-28 15:26:08 UTC 
testuser@home:~$
(gnome-nettool:9757): libglade-WARNING **: could not find signal handler 'gn_quit_app'.

[1]+  Done                    gnome-nettool
testuser@home:~$
Comment 3 Brent Smith (smitten) 2006-01-28 16:10:40 UTC 
Created attachment 58266 [details] [review]
connect the activate signal to gn_quit_app() callback

This connects the activate signal to gn_quit_app() callback for the Quit menu item.

It also removes the <signal> elements from there glade file, since there is no reason to have them there if you are connect the callback from the code.
Comment 4 Rodrigo Moya 2006-01-28 18:26:05 UTC 
Applied to CVS HEAD