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 577353 - use libunique for palimpsest
use libunique for palimpsest
Status: RESOLVED FIXED
Product: gnome-disk-utility
Classification: Core
Component: Disks UI
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-disk-utility-maint
Depends on:
Blocks: 575948
 
 
Reported: 2009-03-30 20:15 UTC by Matthias Clasen
Modified: 2009-03-31 21:58 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
initial unique application support (3.86 KB, patch)
2009-03-31 03:24 UTC, Matthias Clasen
committed Details | Review

Description Matthias Clasen 2009-03-30 20:15:30 UTC
This becomes more important when we allow to launch palimpsest from the smart warning icon.
Comment 1 Matthias Clasen 2009-03-31 03:24:01 UTC
Created attachment 131746 [details] [review]
initial unique application support
Comment 2 Matthias Clasen 2009-03-31 05:31:45 UTC
Small wart: currently, palimpsest seems to use gtk_dialog_run for some of its dialogs (e.g. the "sure you want to delete this partition" dialog). Thus, while the dialog is up, running a second instance of palimpsest will hang waiting for a response from the unique app (which wll come once the dialog is dismissed.
Comment 3 David Zeuthen (not reading bugmail) 2009-03-31 20:20:12 UTC
(In reply to comment #1)
> Created an attachment (id=131746) [edit]
> initial unique application support
> 

Committed. Thanks.
Comment 4 David Zeuthen (not reading bugmail) 2009-03-31 20:32:30 UTC
(In reply to comment #2)
> Small wart: currently, palimpsest seems to use gtk_dialog_run for some of its
> dialogs (e.g. the "sure you want to delete this partition" dialog). Thus, while
> the dialog is up, running a second instance of palimpsest will hang waiting for
> a response from the unique app (which wll come once the dialog is dismissed.

Mmm.. yeah. I don't really have any good ideas here. First of all we can't get rid of modal dialogs; we are messing with data on the hard disks so occasionally we really need to ask confirmation.

Btw, isn't it a libunique bug that it doesn't respond if gtk_dialog_run() is used? I mean, gtk_dialog_run() will still run the main loop and service the D-Bus connection...

Anyway, I think we should just dismiss dialogs (with GTK_RESPONSE_CANCEL) when getting a message from a 2nd palimpsest instance launched with --show-volume or --show-drive. This should also be documented in --help.

Thoughts?
Comment 5 Matthias Clasen 2009-03-31 21:58:40 UTC
We tentatively tracked this down to libdbus holding some lock while calling the pending-call-complete callback, preventing further dbus messages to be processed while we do the gtk_dialog_run. So, this is a dbus bug, and probably not worth working around here.