Re: GnomeICU and playsound.c

[sopwith redhat com (Elliot Lee)]

On Wed, 10 Mar 1999 16:55:47 -0500 (EST), Jacob Berkman
<jberkman@andrew.cmu.edu> wrote:

>On Wed, 10 Mar 1999, Gleef wrote:
>
>> Is the proper fix to nest "#ifdef SOUND" inside the "#ifdef GNOME"s,
>> replace them, or something completely different?
>
>I believe all of the sound stuff is already wrapped in #ifdef SOUND's in
>the rest of GnomeICU.  What the #ifdef GNOME stuff in playsound.c should
>probably be is #ifdef HAVE_ESD.  at the end of gnomeicu.h is this:
>
>#ifdef HAVE_ESD
>#define SOUND
>#define GNOME
>#endif
>
>The problem I have run into is that HAVE_ESD is *not* getting defined in
>config.h, so esd sound ends up not getting compiled in.  Since GnomeICU is
>no longer non-gnome, the #ifdef GNOME should probably become #ifdef
>HAVE_ESD, and that not being defined should be fixed in config.h.in and
>configure.in.

You may just want to use one of the routines listed in
libgnome/gnome-sound.h. If, OTOH, you are trying to have a configurable sound
for an event, using gnome_triggers_do() plus a .soundlist file is the correct
way.

-- Elliot
"In film you will find four basic story lines. Man versus man, man
 versus nature, nature versus nature, and dog versus vampire."
    - Steven Spielberg