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


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.