Bugzilla Bug 182528

Hello, here's a diff of the ebuild that's working well for me, and next is a
revised patch for configure.ac. I've tested with and without all the use flags,
on x86. I think smpeg should probably be a useflag, but didn't want to change
it too much. Lua is packaged with alephone now.

--- /usr/portage/games-fps/alephone/alephone-20061228.ebuild    2007-02-14
11:35:49.000000000 -0500
+++ alephone-20070525.ebuild    2007-06-18 21:00:19.000000000 -0400
@@ -11,13 +11,15 @@
 LICENSE="GPL-2"
 SLOT="0"
 KEYWORDS="amd64 x86"
-IUSE="lua opengl speex"
+IUSE="lua opengl speex mad vorbis sndfile"

-DEPEND="lua? ( dev-lang/lua )
-       opengl? ( virtual/opengl )
+DEPEND="opengl? ( virtual/opengl )
        speex? ( media-libs/speex )
+       mad? ( media-libs/libmad )
+       !mad? ( media-libs/smpeg )
+       vorbis? ( media-libs/libvorbis )
+       sndfile? ( media-libs/libsndfile )
        dev-libs/boost
-       media-libs/smpeg
        media-libs/libsdl
        media-libs/sdl-image
        media-libs/sdl-sound
@@ -38,8 +40,11 @@
 src_compile() {
        egamesconf \
                --disable-dependency-tracking \
-               $(use_enable lua) \
                $(use_enable opengl) \
+               $(use_enable mad) \
+               $(use_enable vorbis) \
+               $(use_enable sndfile) \
+               $(use_enable lua) \
                $(use_enable speex) \
                || die
        emake || die "emake failed"




Reproducible: Always

Steps to Reproduce:

Created an attachment (id=122498) [edit]
alephone-20070525-configure.patch

This is simply the previous patch with the Lua bits removed.

Yeah, smpeg gets linked in if it's installed, libmad or no. Should it get fixed
in the configure patch, same as speex?

Yeah.

Created an attachment (id=122630) [edit]
alephone-20070525.ebuild

Sorry, I know next to nothing about autotools or configure, this is a bit
beyond me.

But from what I can tell, there are bad 'automagic' dependencies on Speex,
smpeg, ALSA, and sdl-image (not a hard dep after all?). What I don't understand
is the line "+CPPFLAGS="$CPPFLAGS -I/usr/include/speex", and if it's required
for the other deps. I'm attaching my last ebuild with an smpeg fix, but as far
as fixing ALSA and sdl-image, if that's desired, I'd prefer someone with more
experience take a look at it.

I did test both with and without smpeg on x86, and it works fine.

Created an attachment (id=122631) [edit]
alephone-20070525-configure.patch

Lastly, it seems sdl-sound is no longer a dependency. :b

To reiterate, these are the differing dependencies:

-lua (now built in)
-sdl-sound
+libmad
+libvorbis
+libsndfile

And these are the automagic dependencies which need fixing AFAICT:

speex
smpeg
alsa
sdl-image

That's it. This version is still working well for me on x86.

Version 0.18.1 (20070819) is now in portage.
Thanks for report and work on ebuild