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 559695 - Opens display before forking
Opens display before forking
Status: RESOLVED FIXED
Product: gnome-settings-daemon
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-settings-daemon-maint
gnome-settings-daemon-maint
Depends on:
Blocks:
 
 
Reported: 2008-11-07 04:58 UTC by Behdad Esfahbod
Modified: 2008-11-10 15:24 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
fork early (4.37 KB, patch)
2008-11-07 19:29 UTC, Behdad Esfahbod
none Details | Review

Description Behdad Esfahbod 2008-11-07 04:58:48 UTC
This is at best discouraged.  Will submit patch to fix.
Comment 1 Behdad Esfahbod 2008-11-07 19:29:09 UTC
Created attachment 122202 [details] [review]
fork early
Comment 2 Jens Granseuer 2008-11-07 19:39:52 UTC
+/* We want the parent process to quit after initializaing all plugins,

s/initializaing/initializing/

+ * process and continue using from the other. So, we just made the

s/using/using it/
s/made/make/

+	gnome_settings_profile_msg ("dettaching daemon");

s/dettaching/detaching/

Indentation is wrong once again.


Otherwise good to go.
Comment 3 Behdad Esfahbod 2008-11-10 15:20:12 UTC
Heh.  That's what happens when I write a patch while waiting for my ride to the airport.  Fixing and committing.
Comment 4 Behdad Esfahbod 2008-11-10 15:24:50 UTC
2008-11-10  Behdad Esfahbod  <behdad@gnome.org>

        * gnome-settings-daemon/main.c (daemon_start), (daemon_detach),
        (daemon_terminate_parent), (main): Fork before gtk_init (bug #559695)