Bugzilla Bug 44809

Hi!

ScummVM 0.6.0, the famous interpreter for SCUMM-games and some others, was released today. Here is an ebuild for it, which has also some other improvements:
I don't know the policy for new USE_EXPANDs, but I have added one:
ENGINES - contains the engines to be built (if empty or undefined all engines get installed), possible values are described in the ebuild

If that is not a good solution then it could also be done by local useflags.

other changes:
- now uses the "flac" use flag and has media-libs/flac as dependency
- changed dependencies to state minimal versions (found in the configure script)
- now uses the "debug" use flag



Reproducible: Always
Steps to Reproduce:
1.
2.
3.

Created an attachment (id=27439) [edit]
ebuild

Where do you see that configure supports flac?

Sorry, my fault. It was in the daily snapshot yesterday, but obviously it has
not made it in the release.

Created an attachment (id=27442) [edit]
ebuild

removed "flac" use flag from ebuild

a hint would be to use "use_enable " for those things instead of the
if-statements

confstr="`use_enable flac` `use_enable zlib` `use_enable debug` `use_enable
mad` `use_enable oggvorbis vorbis`"

Created an attachment (id=27446) [edit]
improved ebuild

I made use of "use_enable" as Spider proposed.

What about the USE_EXPAND, is this ok?

I think all the engines should just be built-in.  If scummvm had broader
application, it would make sense to make the configuration more fine-grained.
It's not going to change the size of the executable by much to just
put all the engines in.  If there's a big demand for it later we can revisit
it.

Created an attachment (id=27448) [edit]
improved ebuild

removed the USE_EXPAND and changed the test of "debug" use flag back to old
"if" solution because there is no "--enable-debug" option.

btw... the use of src_unpack here is redundant. You just duplicate defaults
there.

(oops, missed that in my first lookover)

Created an attachment (id=27449) [edit]
ebuild

redundant src_unpack removed

Bumped the version in CVS.  Thanks for the bug report and ebuild.

A small addition:
According to the README [1] libmpeg2 in version 0.4 or later is preferable for scummvm. So when libmpeg2-0.4.0 bumps to stable, the depend could be set to this.

[1] http://cvs.sourceforge.net/viewcvs.py/scummvm/scummvm/README?rev=release-0-6-0