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 bugzilla.redhat.com.
Bug 196744 - Review Request: chess - 3D chess game
Summary: Review Request: chess - 3D chess 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: Wart
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On: 196710 196740
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-06-26 20:31 UTC by Hans de Goede
Modified: 2007-11-30 22:11 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-07-09 08:35:20 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Hans de Goede 2006-06-26 20:31:36 UTC
Spec URL: http://people.atrpms.net/~hdegoede/chess.spec
SRPM URL: http://people.atrpms.net/~hdegoede/chess-1.0-1.src.rpm
Description:
Beautifull 3D rendered chess game using Ogre. Notice that this needs a
powerful 3D card to be playable on a radeon 92xx or intel 9xx integrated
graphics this is not playable!

Comment 1 Rudolf Kastl 2006-06-28 11:41:46 UTC
what exactly is required? hw accelerated shaders?

Comment 2 Hans de Goede 2006-06-28 11:43:02 UTC
I don't know, something that my radeon 9250 and an intel 945 both lack.


Comment 3 Rudolf Kastl 2006-06-28 11:46:15 UTC
i will take a deeper look on the weekend. a 3d chess game is very nice to have
:) i also saw that you submitted ogre. i was also working on it with wart. but
your package is quite nice. i had a fix for the rpath problem (hack :) ) but the
other fix is indeed nice if it makes the chess game work. nice stuff!

Comment 4 Wart 2006-07-08 20:12:21 UTC
This ran fine for me on FC4-x86_64 with a NVidia 6800 and the binary NVidia
drivers.  Average of ~53 fps.

MUST
====
* Source matches upstream, with the noted exception that the
  copyrighted ttf file has been removed.
* Package and spec named appropriately
* GPL license ok, license file included
* .desktop file installed correctly
* icon cache updated properly
* RPM_BUILD_ROOT cleaned where it ought to be
* Owns directories that it creates
* Does not own directories that it should not
* Builds in mock on FC5-i386, FC5-x86_64
* No need for -devel or -doc subpackages
* No locales
* Not relocatable
* No duplicate %files

SHOULD
======
* Use 'convert -transparent black ...' to make the desktop icon background
  transparent.

APPROVED

Comment 5 Hans de Goede 2006-07-09 08:35:20 UTC
(In reply to comment #4)
> SHOULD
> ======
> * Use 'convert -transparent black ...' to make the desktop icon background
>   transparent.
> 

Good idea, but it cannnot be done (atleast not that way) I created (gimped) the
icon from a screen shot and that black is far from black, also its not one color
and it contains a reflection of the chess piece.

> APPROVED

Thanks a million for this and the other reviews!

Imported & build, closing.



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