Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at

Bug 201153

Summary: Review Request: tuxpuck - 3D Shufflepuck Pong Game
Product: [Fedora] Fedora Reporter: Hugo Cisneiros <hugo>
Component: Package ReviewAssignee: Wart <wart>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: gwync, wart
Target Milestone: ---Flags: wtogami: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-08-07 13:17:33 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 163779    

Description Hugo Cisneiros 2006-08-03 08:41:45 UTC
Spec URL:

TuxPuck is a shufflepuck game written in C using SDL. The player moves a pad
around a board and tries to shoot down the puck through the opponents defense.
Easy to play, difficult to win.

Comment 1 MichaƂ Bentkowski 2006-08-03 10:39:18 UTC
SPEC looks clear, but mock build fails. You have to add BuildRequires:

Comment 2 Wart 2006-08-04 20:39:28 UTC
* rpmlint output clean
* Package and spec file named appropriately
* Spec file legible and in Am. English
* GPL license ok, license file included
* Source matches upstream:
  fc839abc2b1f3eafae397e1ed6487079  tuxpuck-0.8.2.tar.gz
* Builds in mock on FC4, FC5, and FC6 for both i386 and x86_64
* Not relocatable
* No locales
* No shared libs
* RPM_BUILD_ROOT cleaned where necessary
* macro usage consistent
* .desktop file installed correctly
* no -devel package needed
* no need for -doc subpackage
* Does not own directories that it should not.
* Package runs, but won't let me win.  ;)

* Remove the comment about 'doesn't build with optflags', since you are
  building it with %optflags

* Add BR: desktop-file-utils, as mentioned in comment #1

Comment 3 Hugo Cisneiros 2006-08-06 20:16:06 UTC
Thanks for the review guys. Package updated with fixes from Comment #1 and 
Comment #2:

Spec URL:

Comment 4 Wart 2006-08-07 05:39:37 UTC
MUSTFIX items fixed.  Please remember to bump the Release number and update the
%changelog entries during the review.  It makes it easier for the reviewers to
track what's changed.


Comment 5 Hugo Cisneiros 2006-08-07 10:46:42 UTC
I usually only bump the release number when in the reviews, the update to the 
specfile is major and significant :) Thanks for the review.

Comment 6 Hugo Cisneiros 2006-08-07 13:17:33 UTC
Imported and built. Thanks! Closing now.

Comment 7 Gwyn Ciesla 2007-02-26 20:26:22 UTC
Change owner to limb (orphaned)