GNOME Bugzilla – Bug 572913
Migrate from deprecated esound to libcanberra
Last modified: 2009-04-02 17:35:25 UTC
$:andre\> grep -r "include <esd" . ./gok/gok-sound.c:#include <esd.h> $:andre\> grep -r "esd_" . ./gok/gok-sound.c: esd_sample_free (gnome_sound_connection, *((gint *)sample_id) ); ./gok/gok-sound.c: esd_sample_play (gnome_sound_connection, *sample_id); $:andre\> grep -r "esound" . ./configure.in: esound \ ./gok-1.0.pc.in:Requires: libgnomeui-2.0 cspi-1.0 libspi-1.0 libbonobo-2.0 atk gtk+-2.0 gail libwnck-1.0 esound
Can somebody work on this soon, please? This is the very last module in GNOME that directly depends on esound and we want to get rid of this dependency, finally.
Gerd, do you have cycles for this one?
See bug 557921 and bug 558285 for similar issues of other applications. See http://0pointer.de/lennart/projects/libcanberra/ for the spec. Maybe Lennart himself is kind enough to provide a patch if he finds some time...
David: until the end of April I have very little time during the week. I've started to work on a patch some time ago but got sidetracked by #574713. I'll post what I have so far - maybe someone will pick it up. :) If not I'll probably have time to finish it this weekend.
Created attachment 131211 [details] [review] migrate to libcanberra The patch should be finished, except that it doesn't work. I think the problem is that all sound files must be installed in the sound-themes directory. (currently they are in DATADIR/gok)
*** Bug 571489 has been marked as a duplicate of this bug. ***
Created attachment 131617 [details] [review] updated patch - install sounds into freedesktop sound theme - remove test-gok-sound. Playback during distcheck doesn't work because our sounds aren't installed in a theme dir. - 2 new sounds. The old sounds are mono/6kHz and are below the minimum requirements of the spec. I've tried to resample the old files but the resulting quality was very bad.
Created attachment 131618 [details] new sound 1
Created attachment 131619 [details] new sound 2
Thanks a lot. Can we get this in soon for the 2.27.x branch so it receives some testing?
Committed to trunk.