FS#12559 - SDL_sound doesn't build with SMPEG support

Attached to Project: Arch Linux
Opened by Tomi Belan (Tomi) - Wednesday, 24 December 2008, 12:44 GMT
Last edited by Allan McRae (Allan) - Wednesday, 24 December 2008, 15:30 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Allan McRae (Allan)
Architecture All
Severity Low
Priority Normal
Reported Version None
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

SDL_sound supports two libraries that can decode MP3 files: SMPEG, and internal mpglib. It seems that mpglib is broken (I wasn't able to play any MP3 file with "normal" playsound), that's probably a separate upstream bug. This wouldn't be a problem if the SMPEG decoder worked correctly, but it doesn't: SDL_sound's ./configure doesn't look for smpeg.h in /usr/include/smpeg/.

I was able to compile SDL_sound with SMPEG support using:
CFLAGS="$CFLAGS -I/usr/include/smpeg" ./configure
I don't particularly like this method, it looks like a kludge to me. But then, what can I expect from autotools... Maybe ./configure has a better way to add -Ipaths, maybe not.

Once SDL_sound builds with SMPEG support, as it should, smpeg should probably be added to sdl_sound's depends.

Relevant ./configure output:
checking smpeg.h usability... no
checking smpeg.h presence... no
checking for smpeg.h... no
checking for SMPEG_new in -lsmpeg... yes

I use sdl_sound-1.0.3-1 and smpeg-0.4.4-4.

How to reproduce:
* Cause: playsound --decoders | grep SMPEG
* Effect: playsound my_song.mp3
This task depends upon

Closed by  Allan McRae (Allan)
Wednesday, 24 December 2008, 15:30 GMT
Reason for closing:  Fixed
Additional comments about closing:  sdl_sound-1.0.3-2
Comment by Tomi Belan (Tomi) - Wednesday, 24 December 2008, 12:49 GMT
An afterthought. SMPEG support is an optional feature in SDL_sound, and the Arch packagers could disable it. However, since the PKGBUILD doesn't use ./configure --disable-smpeg, I assume that it was supposed to work, and this is a bug.
Comment by Tomi Belan (Tomi) - Wednesday, 24 December 2008, 14:07 GMT
It seems that this is, in part, an upstream bug after all. SMPEG provides smpeg-config, but SDL_sound doesn't use it. (For comparison, SDL_mixer has acinclude/smpeg.m4 which handles it.) Patching the whole build system for Arch would be too much, so the CFLAGS="..." kludge looks like the best available workaround.

Loading...