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 310979 - Configure switch to enable/disable esound support
Configure switch to enable/disable esound support
Status: RESOLVED FIXED
Product: gnome-session
Classification: Core
Component: general
2.11.x
Other Linux
: High trivial
: ---
Assigned To: Session Maintainers
Session Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-07-20 11:19 UTC by Leonardo Boshell
Modified: 2006-11-06 22:49 UTC
See Also:
GNOME target: ---
GNOME version: 2.11/2.12


Attachments
gnome-session-HEAD-esd_switch.patch (1.91 KB, patch)
2005-07-20 11:19 UTC, Leonardo Boshell
committed Details | Review

Description Leonardo Boshell 2005-07-20 11:19:16 UTC
Currently, gnome-session has the necessary pre-processor checks to compile with
or without support for esound in its code. However, esound ends up being
required unconditionally since, for example, configure.in calls AC_MSG_ERROR if
the 'esd' executable is not found.

This could be improved a little bit. I think it would be nice to provide a
switch in 'configure' to enable/disable support for esound. I've created a patch
for this against CVS HEAD, which I've tested it without problems so far.

With the patch, the default behaviour is to enable esound support, but it gives
the option to disable it. I'll attach it here in case there's interest in this.
Thanks.
Comment 1 Leonardo Boshell 2005-07-20 11:19:50 UTC
Created attachment 49451 [details] [review]
gnome-session-HEAD-esd_switch.patch
Comment 2 Vincent Untz 2006-11-06 22:49:33 UTC
Finally committed. Sorry for taking so long.