Spec URL: http://people.atrpms.net/~hdegoede/scorchwentbonkers.spec SRPM URL: http://people.atrpms.net/~hdegoede/scorchwentbonkers-1.1-1.fc7.src.rpm Description: As the name suggests, Scorch Went Bonkers is a remake of the old PC classic. However, many things were changed and the type of fun delivered by the game is different. Where Scorched Earth puts emphasis on tactics and careful calculations, SWB requires quick thinking, perfect timing and only one finger for controlling your tank. The game is real-time instead of turn based.
Notice that this needs the still to be reviewed jpgalleg lib, whose review is bug 227198
Indeed, don't forget to add: BuildRequries: jpgalleg-devel
Fails to link, appears you will also need to add: BuildRequires: libGLU-devel
Like the music, but when I go into battle mode and select one player as the computer the game crashes. $ swb Shutting down Allegro due to signal #11 Segmentation fault
When shooting off edge of screen, bullets do not appear in correct position on opposite side of screen (that is, the bullets should re-appear at the opposite edge of the screen, not in the middle of the screen).
Hmm, ignore comment #5 as I cannot reproduce this now. It appears the bullets bounce off the edges, but I could have sworn it appeared differently before. Oh well, I could have just looked at it incorrectly.
(In reply to comment #4) > Like the music, but when I go into battle mode and select one player as the > computer the game crashes. > > $ swb > Shutting down Allegro due to signal #11 > Segmentation fault > Nasty FP divide by zero leading to NAN leading to other funnies only happening on x86_64 bug -> fixed. I also added the report missing BR's, new version here: Spec URL: http://people.atrpms.net/~hdegoede/scorchwentbonkers.spec SRPM URL: http://people.atrpms.net/~hdegoede/scorchwentbonkers-1.1-2.fc7.src.rpm
==== REVIEW CHECKLIST ==== - rpmlint output clean - package named according to package naming guideliens - spec file name matches %{name} - package meets packaging guidelines - licensed with open source compatible license - license matches actual license - license included in %doc - spec written in American english - spec file legible - sources match upstream dfaf1a509d2776edf983cd5e1540df95 - package successfully compiles and builds on FC-6 x86_64 - all build dependencies listed in BR - no locales - no shared libraries - package is not relocatable - package owns all directories it creates - all directories it does not create are owned pulled in from Requires - no duplicates in %files - file permissions set properly - package contains proper %clean - macro usage is consistent - contains code - no large documentation - files in %doc do not affect runtime - no header files - no static libraries - no pkgconfig files - no need for devel subpackage - no libtool archives - contains proper desktop file - package does not own files or directories owned by other packages *** APPROVED ***
New Package CVS Request ======================= Package Name: scorchwentbonkers Short Description: Realtime remake of Scorched Earth Owners: j.w.r.degoede Branches: FC-6 devel InitialCC: <empty>
done
Thanks for the review! Imported and build, closing.