Bug 286631 - Review Request: slingshot - A Newtonian strategy game
Summary: Review Request: slingshot - A Newtonian strategy game
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Hans de Goede
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2007-09-11 18:32 UTC by Gwyn Ciesla
Modified: 2007-11-30 22:12 UTC (History)
3 users (show)

Fixed In Version: 0.8.1p-1.fc7
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2007-09-18 03:18:06 UTC
Type: ---
hdegoede: fedora-review+
kevin: fedora-cvs+

Attachments (Terms of Use)

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.

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