Bug 286631

Summary: Review Request: slingshot - A Newtonian strategy game
Product: [Fedora] Fedora Reporter: Gwyn Ciesla <gwync>
Component: Package ReviewAssignee: Hans de Goede <hdegoede>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, hdegoede, notting
Target Milestone: ---Flags: hdegoede: fedora-review+
kevin: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: 0.8.1p-1.fc7 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-09-18 03:18:06 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Gwyn Ciesla 2007-09-11 18:32:20 UTC
Spec URL: http://zanoni.jcomserv.net/fedora/slingshot/slingshot.spec
SRPM URL: http://zanoni.jcomserv.net/fedora/slingshot/slingshot-0.8.1p-1.fc7.src.rpm
Description: Slingshot is a two dimensional, turn based simulation-strategy game
set in the gravity fields of several planets. It is a highly
addictive game, and never the same from round to round due to its
randomly generated playing fields.

Comment 1 Hans de Goede 2007-09-11 18:55:36 UTC
Shall we exchange reviews again? As usual pick any of mine from the Games SIG page.

Comment 2 Hans de Goede 2007-09-11 21:55:41 UTC
Fully reviewed, all MUST items passed, and it works fine on x86_64 and seems
like a good way to waist some time :)

Comment 3 Gwyn Ciesla 2007-09-11 22:09:04 UTC
Thanks much.

I was afraid you'd never finish the review if you played it.  I'm amazed I was
able to break away long enough to package the thing. :)

New Package CVS Request
Package Name: slingshot
Short Description: A Newtonian strategy game
Owners: limb
Branches: FC-6 F-7
Cvsextras Commits: yes

Comment 4 Ignacio Vazquez-Abrams 2007-09-12 00:51:38 UTC
Why does the package include a copy of FreeSansBold.ttf instead of requiring
freefont and using the file in there?

Comment 5 Gwyn Ciesla 2007-09-12 00:58:29 UTC
Good question.  I didn't know it was already included.  Would it be better to
replace the included copy with a Requires: and a symlink?  Or, instead of a
symlink, I could patch the slingshot.py.  Which is preferred?

Comment 6 Ignacio Vazquez-Abrams 2007-09-12 01:47:32 UTC
Well, patching will use up one less inode on the disk and save a bit of space in
the package. Other than that I don't think it really matters.

Comment 7 Gwyn Ciesla 2007-09-12 02:17:35 UTC
Complete.  It will be imported with that change.

Comment 8 Kevin Fenzi 2007-09-12 15:54:28 UTC
cvs done.

Comment 9 Fedora Update System 2007-09-18 03:18:05 UTC
slingshot-0.8.1p-1.fc7 has been pushed to the Fedora 7 stable repository.  If problems still persist, please make note of it in this bug report.