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 523743 - don't check /desktop/gnome/sound/enable_esd, if configure --disable-esd
don't check /desktop/gnome/sound/enable_esd, if configure --disable-esd
Status: RESOLVED FIXED
Product: gnome-settings-daemon
Classification: Core
Component: plugins
2.22.x
Other All
: Normal normal
: ---
Assigned To: gnome-settings-daemon-maint
gnome-settings-daemon-maint
Depends on:
Blocks:
 
 
Reported: 2008-03-21 22:03 UTC by Alexey Shabalin
Modified: 2008-03-24 17:18 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22

Attachments
always check in /desktop/gnome/sound/enable_esd (835 bytes, patch)
2008-03-21 22:05 UTC, Alexey Shabalin 		committed Details | Review

Description Alexey Shabalin 2008-03-21 22:03:28 UTC 
Please describe the problem:
gnome-setting-daemon compiled with "configure --disable-esd".
In plugins/sound/gsd-sound-manager.c:

#ifdef HAVE_ESD 
        enable_sound = gconf_client_get_bool (client, "/desktop/gnome/sound/enable_esd", NULL);
#else 
        enable_sound = TRUE;
#endif 

and next if (enable_sound)-> start_gnome_sound -> "esd -nobeeps"ALWAYS


I to be in want of compile with "configure --disable-esd" and don't run esd, if /desktop/gnome/sound/enable_esd = FALSE



Steps to reproduce:
1. 
2. 
3. 


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Alexey Shabalin 2008-03-21 22:05:36 UTC 
Created attachment 107763 [details] [review]
always check in /desktop/gnome/sound/enable_esd
Comment 2 Jens Granseuer 2008-03-22 09:56:18 UTC 
No, "esd" is started on purpose here. See bug 518075 for an explanation.
Comment 3 Alexey Shabalin 2008-03-22 22:35:42 UTC 
1. configure --disable-esd
2. set FALSE in gconf desktop/gnome/sound/enable_esd
And i don't want start esd really. But esd startd, started, and again started.
How disable esd?
Comment 4 Jens Granseuer 2008-03-24 17:18:14 UTC 
Uninstall it?

Although now that "esd" can mean both esd or PulseAudio, you probably have a point, so ...

2008-03-24  Jens Granseuer  <jensgr@gmx.net>

        Patch by: Alexey Shabalin <a.shabalin@gmail.com>
                                                      
        * plugins/sound/gsd-sound-manager.c: (apply_settings): now that
        "starting esd" can mean either esd or PulseAudio, check the GConf
        setting for starting the sound server even when esd is disabled
        (bug #523743)