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 226729 (duel3)

Summary: Review Request: duel3 - One on one spaceship duel in a 2D arena
Product: [Fedora] Fedora Reporter: Hans de Goede <hdegoede>
Component: Package ReviewAssignee: Christopher Stone <chris.stone>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: chris.stone
Target Milestone: ---Keywords: Reopened
Target Release: ---Flags: chris.stone: fedora-review+
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-02-19 17:57:29 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 Hans de Goede 2007-02-01 09:10:56 UTC
Spec URL:
The sudden attack from the Martain Rim miners caught the Earth by suprise,
there was no way the meager Earth Space Fleet could defend themselves. The
miners attacked, and eliminated their enemies, and then returned to the
asteroid belt. However, Earth could not accept such an embarrassing defeat. The
military developed new space fighters, and trained several squadrons of elite
pilots. The task force was then deployed against the miners. These trained
pilots utterly defeated the miners in a matter of weeks, and the first space
war in human history was finished.

The military, however, now had a new problem on their hands. These new elite
pilots were becoming restless, and there was no way for them to test their
skills. The military dare not disband the force, or let their skills dull, so
the Duel Combat League was formed. The newly formed league quickly became the
premier entertainment form on the planet, and the military's largest source of

Take control of a Duel fighter, and test your skills againt your opponents and
the arena itself in fast-paced space combat.


Notice that the music in the game comes from here:
As is stated in the readme and does not seem to come with any license of its own, I've contacted the author of the music for further license information.

Comment 1 Hans de Goede 2007-02-01 12:09:51 UTC
The music issue has been cleared, I got the following answer from the author of
the music:


the first thing I have to say is that I'm spanish, so I hope that my english
will be fine.

thanks for your e-mail. These songs can be used freely. The only thing I want is
to be credited on the game as "Antonio Salgado" that  is my real name.

When I'll arrive at home I'll try the game, It looks good.

thanks again.

see you!!"

So only proper crediting is needed, I'll fix this with the next release,
together with any other issues which come up during review.

Comment 2 Christopher Stone 2007-02-01 16:40:42 UTC
Standard review comment for Hans' packages: icon-theme??  ;-)

Comment 3 Hans de Goede 2007-02-02 07:33:24 UTC
AHHH , me stupid yes I forgot that as usual, will I ever learn? I'll fix that
together with a note about the music license and any other needed fixes when a
full review is done. Unless the reviewer wants a fixed version before beginning
in that case let me know.

Comment 4 Christopher Stone 2007-02-02 17:10:52 UTC
No it's quite alright.  I left this bug open for a day or so incase Jochen
wanted to review it, but since no one has taken it I will.

Comment 5 Christopher Stone 2007-02-02 18:05:09 UTC
- rpmlint output clean
- package named according to package naming guidelines
- spec file name matches %{name}
- package meets packaging guidelines
- package licensed with open source compatible license
- license tag matches actual license
X license file included in %doc
- spec written in American english
- spec file is legible
- sources match upstream
- successfully compiles and builds on FC-6 x86_64
- all build dependencies listed in BR
- no locales
- no shared libraries
- not relocatable
- package owns all directories it creates
X package does not bring in all Requires for dirs it does not own
- no duplicates in %files
- file permissions set properly
- contains proper %clean section
- macro usage consistent
- package contains code
- no large documentation
- %doc does not affect runtime
- no header files or static libs
- no pkgconfig files
- no library files with suffix
- no need for devel subpackage
- no libtool archives
- contains proper desktop file
- does not own files or directories owned by other packages

==== MUST FIX ====
- Requires icon cache package to pull in icon dir
- Include artwork license in %doc

Comment 6 Hans de Goede 2007-02-03 10:49:59 UTC
(In reply to comment #5)
> ==== MUST FIX ====
> - Requires icon cache package to pull in icon dir

> - Include artwork license in %doc

I assume you mean the music license as the rest of the artwork is under the same
license as the source -> done.

New version here:
Spec URL:

Comment 7 Christopher Stone 2007-02-03 16:34:32 UTC
Hans, please in the future follow the new guidelines outlined here:

And ASSIGN this bug back to me when the ball is back in my court.  I happened to
catch this e-mail by chance since it wasnt ASSIGNED to me.

I know this is a total pain, please comment on fedora-packaging if you do not
like the new process (I did).

Anyway, all must items fixed.

Comment 8 Bill Nottingham 2007-02-05 22:05:56 UTC
Package pre-import admin stuff done. Feel free to import.

Comment 9 Hans de Goede 2007-02-07 15:29:02 UTC
Imported and build, closing.

Comment 10 Mamoru TASAKA 2007-02-19 17:56:29 UTC
(Just fixing...)