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

http://bugzilla.rpmfusion.org/show_bug.cgi?id=1122 --- Comment #14 from Julian Sikorski <belegdol(a)gmail.com&gt; 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.