Bug 227676 - (scorchwentbonkers) Review Request: scorchwentbonkers - Realtime remake of Scorched Earth
Review Request: scorchwentbonkers - Realtime remake of Scorched Earth
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Christopher Stone
Fedora Package Reviews List
:
Depends On: jpgalleg
Blocks: FE-ACCEPT
  Show dependency treegraph
 
Reported: 2007-02-07 10:13 EST by Hans de Goede
Modified: 2007-11-30 17:11 EST (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-03-15 08:00:56 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
chris.stone: fedora‑review+
petersen: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Hans de Goede 2007-02-07 10:13:29 EST
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.
Comment 1 Hans de Goede 2007-02-07 10:16:35 EST
Notice that this needs the still to be reviewed jpgalleg lib, whose review is
bug 227198
Comment 2 Christopher Stone 2007-03-12 17:35:15 EDT
Indeed, don't forget to add:

BuildRequries: jpgalleg-devel
Comment 3 Christopher Stone 2007-03-12 17:43:56 EDT
Fails to link, appears you will also need to add:

BuildRequires: libGLU-devel
Comment 4 Christopher Stone 2007-03-12 18:09:39 EDT
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
Comment 5 Christopher Stone 2007-03-12 18:28:14 EDT
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).
Comment 6 Christopher Stone 2007-03-12 19:17:41 EDT
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.
Comment 7 Hans de Goede 2007-03-13 09:13:27 EDT
(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
Comment 8 Christopher Stone 2007-03-14 20:51:45 EDT
==== 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 ***
Comment 9 Hans de Goede 2007-03-15 03:57:15 EDT
New Package CVS Request
=======================
Package Name:      scorchwentbonkers
Short Description: Realtime remake of Scorched Earth
Owners:            j.w.r.degoede@hhs.nl
Branches:          FC-6 devel 
InitialCC:         <empty>
Comment 10 Jens Petersen 2007-03-15 04:05:30 EDT
done
Comment 11 Hans de Goede 2007-03-15 08:00:56 EDT
Thanks for the review!

Imported and build, closing.

Note You need to log in before you can comment on or make changes to this bug.