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 525859 - gnome-settings-daemon needs to export a stop API
gnome-settings-daemon needs to export a stop API
Status: RESOLVED OBSOLETE
Product: gnome-settings-daemon
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-settings-daemon-maint
gnome-settings-daemon-maint
: 512914 (view as bug list)
Depends on:
Blocks: 522017
 
 
Reported: 2008-04-02 22:30 UTC by Rob Bradford
Modified: 2008-05-27 19:58 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22


Attachments
Patch adding 'Stop' dbus method (3.17 KB, patch)
2008-04-02 22:57 UTC, Rob Bradford
rejected Details | Review

Description Rob Bradford 2008-04-02 22:30:10 UTC
+++ This bug was initially created as a clone of Bug #522017 +++

gnome-session needs to stop gnome-settings-daemon cleanly, at the moment it doesn't, esd is still using the sound device after logout and that breaks sound for the next logins

====

In order to achieve the goal for the original bug gnome-settings-daemon needs a DBUS method exported so it can know when to tidy up.

I'm going to attach a patch.
Comment 1 Rob Bradford 2008-04-02 22:57:28 UTC
Created attachment 108511 [details] [review]
Patch adding 'Stop' dbus method

Proposed patch.

Also some plugins, in particular the sound one, need to have their deactivate function worked on.
Comment 2 Jens Granseuer 2008-04-04 17:27:46 UTC
(In reply to comment #1)
> Also some plugins, in particular the sound one, need to have their deactivate
> function worked on.

Make that all plugins. That's an issue that really needs to be fixed before we can even think about using module shutdown.
Comment 3 Jens Granseuer 2008-04-09 16:16:40 UTC
*** Bug 512914 has been marked as a duplicate of this bug. ***
Comment 4 Jens Granseuer 2008-04-29 20:17:43 UTC
From what I have gathered by now, the new g-s explicitly does *not* want to introduce special casing for g-s-d. Has this changed in some way? Otherwise I don't see how this is going to help. Assuming g-s terminates apps with SIGTERM on shutdown, how about having g-s-d handle that by shutting down cleanly?
Comment 5 Bastien Nocera 2008-04-30 10:37:13 UTC
We'd obviously also need to make sure that gnome-session doesn't just send SIGTERM and go away, otherwise the display will be taken from under us, and most apps will exit straight away (such as PulseAudio with the x11 module loaded).
Comment 6 Jens Granseuer 2008-05-27 19:58:41 UTC
Obsoleted by bug 522017.