[Bug 1122] Review request: mame - Multiple Arcade Machine Emulator

[RPM Fusion Bugzilla]

http://bugzilla.rpmfusion.org/show_bug.cgi?id=1122





--- Comment #14 from Julian Sikorski <belegdol at gmail.com>  2010-03-21 14:34:40 ---
I asked upstream to include ppc64 detection a while ago, I guess they forgot. I
think I can make a patch, I think. The problem is the lack of ppc64 machine to
test, since as oget said, it's not allowed to use koji for that.
Same goes for internal error, I can't test it until the package is imported.

The main difference for inis is that they enable mouse by default, which I
don't think is a particularly good idea - IIRC it gets enabled for gun games
anyway.
When it comes to man pages, I'm against it, unless we find a way to generate
them dynamically at build time. Otherwise, they are pretty much bound to get
out of sync at some point, spelling trouble.

VERBOSE=1 could be added, sure, but this makes build logs _huge_. In the past I
only enabled it when something went wrong. And I usually only changed @gcc to
gcc.

Dropping suffix64 is a good idea, I don't think that anybody would want to
install 32 and 64-bit versions of MAME in parallel.

Other arches are not supported anyway, and there is likely to be more trouble
than just the wrong flags. In case someone wants to build mame on them, I'm
happy to push this through RB/mamedev.

We can move the ctrlr files, but this has to be coordinated with frontend
maintainers, so I'd rather wait after the package is imported.

Having said that:
Spec URL: http://homepage.hispeed.ch/belegdol/mame/mame.spec
SRPM URL: http://homepage.hispeed.ch/belegdol/mame/mame-0.137-3.fc13.src.rpm

Changes:
- Dropped suffix64
- Added ppc64 autodetection support
- Re-diffed the fortify patch


-- 
Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.