Bugzilla Bug 131418

Hi,
StepMania 3.9 OGG support on my AMD64 system seems broken even if I've compiled
it with "vorbis" USE flag.

When trying to play an OGG file I get :
WARNING: RageSound::Load: error opening sound "......ogg": Unknown file format
in the console.

Thanks.
Nicolas.

pcnicolas nicolas # emerge --info
Portage 2.1_pre9-r4 (default-linux/amd64/2005.1, gcc-3.4.5, glibc-2.3.6-r2,
2.6.15-gentoo-r1 x86_64)
=================================================================
System uname: 2.6.15-gentoo-r1 x86_64 AMD Athlon(tm) 64 X2 Dual Core 3800+
Gentoo Base System version 1.12.0_pre16
dev-lang/python:     2.3.5-r2, 2.4.2-r1
sys-apps/sandbox:    1.2.17
sys-devel/autoconf:  2.13, 2.59-r7
sys-devel/automake:  1.4_p6, 1.5, 1.6.3, 1.7.9-r1, 1.8.5-r3, 1.9.6-r1
sys-devel/binutils:  2.16.1-r1
sys-devel/libtool:   1.5.22
virtual/os-headers:  2.6.11-r3
ACCEPT_KEYWORDS="amd64 ~amd64"
AUTOCLEAN="yes"
CBUILD="x86_64-pc-linux-gnu"
CFLAGS="-mtune=k8 -O2 -pipe"
CHOST="x86_64-pc-linux-gnu"
CONFIG_PROTECT="/etc /usr/kde/2/share/config /usr/kde/3.5/env
/usr/kde/3.5/share/config /usr/kde/3.5/shutdown /usr/kde/3/share/config
/usr/lib64/mozilla/defaults/pref /usr/share/X11/xkb /usr/share/config
/var/qmail/control"
CONFIG_PROTECT_MASK="/etc/eselect/compiler /etc/gconf /etc/terminfo /etc/env.d"
CXXFLAGS="-mtune=k8 -O2 -pipe"
DISTDIR="/usr/portage/distfiles"
FEATURES="autoconfig distlocks metadata-transfer sandbox sfperms strict"
GENTOO_MIRRORS="http://distfiles.gentoo.org
http://distro.ibiblio.org/pub/linux/distributions/gentoo"
LANG="fr_FR@euro"
LINGUAS="fr"
MAKEOPTS="-j3"
PKGDIR="/usr/portage/packages"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR="/usr/portage"
PORTDIR_OVERLAY="/usr/portage_overlay"
SYNC="rsync://rsync.gentoo.org/gentoo-portage"
USE="amd64 X aac acpi alsa apache2 apm artworkextra avi berkdb bitmap-fonts
cairo cdr cli crypt cups dbus doc dri dvdr eds emboss encode ffmpeg firefox
flac foomaticdb fortran gif glitz gnome gpm gstreamer gtk gtk2 hal howl imlib
ipv6 isdnlog jpeg kde lzw lzw-tiff mad mono mp3 mpeg musepack musicbrainz mysql
nautilus ncurses nls nsplugin ogg opengl pam pcre pdflib perl png ppds pppd
python qt quicktime readline reflection sdl session simplexml spell spl ssl
tcpd theora tiff truetype-fonts type1-fonts usb vcd visualization vorbis
xcomposite xine xorg xpm xv xvid zlib elibc_glibc kernel_linux linguas_fr
userland_GNU"
Unset:  ASFLAGS, CTARGET, EMERGE_DEFAULT_OPTS, INSTALL_MASK, LC_ALL, LDFLAGS


pcnicolas nicolas # equery uses ffmpeg
[ Searching for packages matching ffmpeg... ]
[ Colour Code : set unset ]
[ Legend        : Left column  (U) - USE flags from make.conf                  
  ]
[                  : Right column (I) - USE flags packages was installed with ]
[ Found these USE variables for media-video/ffmpeg-0.4.9_p20060302 ]
 U I
 - - a52      : Enables support for decoding ATSC A/52 streams used in DVD
 + + aac      : Enables support for MPEG-4 AAC Audio
 - - altivec  : Adds support for optimizations for G4 and G5/ppc970 processors
 - - amr      : Enables Adaptive Multi-Rate Audio support
 - - debug    : Tells configure and the makefiles to build for debugging.
Effects vary across packages, but generally it will at least add -g to CFLAGS.
Remember to set FEATURES=nostrip too
 + + doc      : Adds extra documentation (API, Javadoc, etc)
 - - dts      : Enables libdts (DTS Coherent Acoustics decoder) support
 + + encode   : Adds support for encoding of audio or video files
 - - ieee1394 : Enable FireWire/iLink IEEE1394 support (dv, camera, ...)
 + + imlib    : Adds support for imlib, an image loading and rendering library
 - - mmx      : Adds support for optimizations for Pentium MMX and Athlon class
processors
 - - network  : Enables network streaming support
 + + ogg      : Adds support for the Ogg container format (commonly used by
Vorbis, Theora and flac)
 - - oss      : Adds support for OSS (Open Sound System)
 + + sdl      : Adds support for Simple Direct Layer (media library)
 - - test     : Workaround to pull in packages needed to run with
FEATURES=maketest / FEATURES=test
 + + theora   : Adds support for the Theora Video Compression Codec
 - - threads  : Adds threads support for various packages. Usually pthreads
 - - truetype : Adds support for FreeType and/or FreeType2 fonts
 - - v4l      : Enables video4linux support
 + + vorbis   : Adds support for the OggVorbis audio codec
 - - x264     : Enables h264 encoding using x264
 + + xvid     : Adds support for xvid.org's open-source mpeg-4 codec
 + + zlib     : Adds support for zlib (de)compression


pcnicolas nicolas # equery uses libvorbis
[ Searching for packages matching libvorbis... ]
[ Colour Code : set unset ]
[ Legend        : Left column  (U) - USE flags from make.conf                  
  ]
[                  : Right column (I) - USE flags packages was installed with ]
[ Found these USE variables for media-libs/libvorbis-1.1.2 ]
 U I
 - + aotuv : Allow users to enable aoTuV encoder enhancements


pcnicolas nicolas # equery uses libmad
[ Searching for packages matching libmad... ]
[ Colour Code : set unset ]
[ Legend        : Left column  (U) - USE flags from make.conf                  
  ]
[                  : Right column (I) - USE flags packages was installed with ]
[ Found these USE variables for media-libs/libmad-0.15.1b-r1 ]
 U I
 - - debug : Tells configure and the makefiles to build for debugging. Effects
vary across packages, but generally it will at least add -g to CFLAGS. Remember
to set FEATURES=nostrip too

I confirm the bug: I'm suffering this problem with an athlon-xp.

# emerge --info
Portage 2.1_pre9-r4 (default-linux/x86/2006.0, gcc-4.1.0, glibc-2.4-r1,
2.6.16.11 i686)
=================================================================
System uname: 2.6.16.11 i686 AMD Athlon(tm) XP 1600+
Gentoo Base System version 1.12.0_pre18
dev-lang/python:     2.4.2-r1
sys-apps/sandbox:    1.2.17
sys-devel/autoconf:  2.13, 2.59-r7
sys-devel/automake:  1.4_p6, 1.5, 1.6.3, 1.7.9-r1, 1.8.5-r3, 1.9.6-r2
sys-devel/binutils:  2.16.1-r2
sys-devel/libtool:   1.5.22
virtual/os-headers:  2.6.11-r5
ACCEPT_KEYWORDS="x86 ~x86"
AUTOCLEAN="yes"
CBUILD="i686-pc-linux-gnu"
CFLAGS="-march=athlon-xp -O3 -pipe -ftree-vectorize -ftree-vectorizer-verbose=2
-fomit-frame-pointer"
CHOST="i686-pc-linux-gnu"
CONFIG_PROTECT="/etc /usr/kde/2/share/config /usr/kde/3.5/env
/usr/kde/3.5/share/config /usr/kde/3.5/shutdown /usr/kde/3/share/config
/usr/share/X11/xkb /usr/share/config /var/qmail/control"
CONFIG_PROTECT_MASK="/etc/eselect/compiler /etc/gconf /etc/revdep-rebuild
/etc/terminfo /etc/env.d"
CXXFLAGS="-march=athlon-xp -O3 -pipe -ftree-vectorize
-ftree-vectorizer-verbose=2 -fomit-frame-pointer -fvisibility-inlines-hidden"
DISTDIR="/usr/portage/distfiles"
FEATURES="autoconfig distlocks metadata-transfer sandbox sfperms strict"
GENTOO_MIRRORS="http://distfiles.gentoo.org
http://distro.ibiblio.org/pub/linux/distributions/gentoo"
LANG="en_US.UTF-8"
LDFLAGS="-O1"
PKGDIR="/usr/portage/packages"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR="/usr/portage"
PORTDIR_OVERLAY="/usr/local/portage"
SYNC="rsync://rsync.gentoo.org/gentoo-portage"
USE="x86 3dnow 3dnowex 3dnowext 3ds X a52 aac aalib acpi alsa amr ao aotuv apm
arts artworkextra asm async atm audiofile avi beepmp bidi bitmap-fonts
bluetooth bzip2 cairo canvas cdda cddb cdio cdparanoia cjk cli crypt cups curl
custom-cflags dbus dga drac dri dts dv dvb dvd dvdread edl eds emboss encode
esd evo extras faad fame fbcon fbdev ffmpeg firefox flac foomaticdb fortran
freewnn gb gcj gdbm ggi gif gimp gimpprint glx gmp gnome gnomedb gnutls gpm
gstreamer gtk gtk2 hal hbci hou httpd id3 imagemagick imlib imlib2 ipv6 irc
irmc isdnlog jabber jack java jce joystick jpeg jpeg2k kde kqemu ladcca lame
ldap libcaca libg++ libwww live love lzo mad maildir maps matroska mbox migemo
mikmod mjpeg mmap mmx mmx2 mmxext mng mod monkey motif mozdevelop moznocompose
moznoirc moznomail mozsvg mp3 mp4live mpeg mpeg2 mpeg4 mpi mplayer mppe-mppc
msn mule musepack music nas ncurses net netboot netjack network nls nocd nodrm
nowin nptl nptlonly nsplugin ntfs numeric nvidia objc offensive ogg openal
opengl oss pam pcre pdflib perl player png portaudio ppds pppd python qt
quicktime readline real reflection reiserfs rss rtc screen sdl session silc
silverxp skey slang sndfile socks5 softmmu sou sounds sox speex spell spl sse
ssl stream svg svga tcltk tcpd tga theora tiff timidity transcode truetype
truetype-fonts type1 type1-fonts udev unicode userlocales utf8 uudeview v4l
v4l2 vcd vhosts videos vidix vim vim-pager vlm vorbis win32codecs wmf wxwindows
x264 xanim xface xfs xml xmms xorg xosd xprint xv xvid xvmc zlib elibc_glibc
kernel_linux userland_GNU"
Unset:  ASFLAGS, CTARGET, EMERGE_DEFAULT_OPTS, INSTALL_MASK, LC_ALL, LINGUAS,
MAKEOPTS

I can confirm this too.  I had a quick look around, and although stepmania is
being linked against ogg/vorbis libs, it only tries to detect mp3 and wav
filetypes.  There aren't many changes in the code between -rc3 and release
version so I'm not sure why this is happening.  I looked at the patches, and
while the alsa patch looks totally wrong, removing it doesn't resolve this
issue.

OK, found the problem.  It's a duff configure file produced by autoconf.  If
you do ./configure --with-vorbis it skips the vorbis/ogg tests.  If I
understood how autotools worked I would edit aclocal.m4 to fix the test, but I
don't.  I have, however, changed the ebuild to use the old way:

- $(use_with vorbis) \
+ $(use vorbis || echo --without-vorbis) \

The configure tests will then be run, and egg files will then play as normal (I
have tested this fully, but a glance at configure output will also confirm).

However, I still have doubts about the alsa patch and urge someone to look at
it.  It looks to me as if it will SetSWParams only if SetHWParams succeeds,
which is the opposite of what it used to do.  It seems to me you would likely
want to set software parameters if setting hardware parameters failed! I'm not
really into the whole C++ thing though, so I leave it up to someone else to
determine the right course of action.

I would suggest to people who need this patch that they probably have a broken
(perhaps superfluous) asoundrc, or should set SoundDrivers=ALSA-sw in their
Data/StepMania.ini

The previously listed solution is NOT the proper way to fix this. This is an
upstream problem with the way one of the .m4 files is written. I am attaching a
patch to fix this problem in the source tree.

Created an attachment (id=86377) [edit]
Patch to fix vorbis support in stepmania 3.9

Created an attachment (id=86506) [edit]
Fixed Sean's patch.

Changed the patch slightly to actually point to the right file (aclocal.m4, not
audio.m4).  I don't know if this is because Sean is using another version of
stepmania or what; but the change I've made to the patch is right for the
latest version in portage (3.9).

Otherwise I've tested the patch with USE=vorbis and USE=-vorbis and it works
fine.

"autoconf || die 'autoconf failed'" will need to be added after the epatch
line, and I still suggest that the alsa patch is removed for the reasons I gave
above. :)

Re-assign wrt Bug 133117. Please, remove warpzero from metadata.xml.

Some action on this package would be good.  The version in portage is still
broken, and this patch fixes the issue in the same way it's fixed upstream in
cvs.  It just needs putting into portage (by anyone, I guess, since there's no
maintainer for this package now).

Fixed in portage.
alsa patch has not been removed, waiting for feedback from the guy who wrote,
CC'ed.
Please damien, check the suggestion at comment #3 and remove the alsa patch. If
that work, tell us

(In reply to comment #9)
> Fixed in portage.
> alsa patch has not been removed, waiting for feedback from the guy who wrote,
> CC'ed.
> Please damien, check the suggestion at comment #3 and remove the alsa patch. If
> that work, tell us
> 

I can't compile stepmania anymore (SDL_EventMask error, i'll see it later)(And
I can't test the vorbis/autools trick too) , but with "SoundDrivers=Alsa-sw", I
have no warnings anymore, so it should be OK.

The change in the ebuild from comment #3 is working for me (and without alsa
"patch").

I submitted a bug about the sdl error ( bug #135440 )