Bug 210632 - Review Request: fbg - Falling Block Game
Summary: Review Request: fbg - Falling Block Game
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Michał Bentkowski
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-10-13 13:47 UTC by Hans de Goede
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-10-14 11:47:39 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Hans de Goede 2006-10-13 13:47:05 UTC
Spec URL: http://people.atrpms.net/~hdegoede/fbg.spec
SRPM URL: http://people.atrpms.net/~hdegoede/fbg-0.9-1.fc6.src.rpm
Description:
Move and rotate variously shaped falling blocks to make them form a complete
row. Once a complete row has formed it will disappear. If you make
incomplete rows the screen will fill up and when it reaches the top the game
is over.

Comment 1 Michał Bentkowski 2006-10-13 14:56:02 UTC
MUST items:
 * rpmlint doesn't show anything
 * package is named well
 * spec file name is good
 * package meets Packaging Guidelines
 * package is licensed with an GPL open-source compatible license
 * License field in spec file doesn't match actual license
 * license file is included in %doc
 * md5sums are matching (82db64d84b6172f5676fcff69d533881)
 * package successfully compiles on x86_64
 * BuildRequires listed well (mock builds fine)
 * no locales
 * proper %post and %postun sections
 * not relocatable
 * package owns directories well
 * no duplicates in %files
 * every %files section includes %defattr
 * proper %clean section
 * macros used well
 * .desktop file is present

THINGS to do:
 * change spec's License field to GPL

Fix thing mentioned above and package will be approved,

Comment 2 Hans de Goede 2006-10-14 06:49:06 UTC
Woops, good catch. I used a spec from a previous package for this and that
stayed in.

Here is a version with the license tag fixed:
Spec URL: http://people.atrpms.net/~hdegoede/fbg.spec
SRPM URL: http://people.atrpms.net/~hdegoede/fbg-0.9-2.fc6.src.rpm


Comment 3 Michał Bentkowski 2006-10-14 10:08:26 UTC
You haven't updated a spec file but a SRPM looks good.

So package is APPROVED.

Comment 4 Hans de Goede 2006-10-14 11:47:39 UTC
Oops, you're right I forgot to upload the spec. Thanks for looking at the SRPM.
I've imported and build the correct/updated srpm, closing.

Thanks!



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