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 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 Review | Assignee: | Christopher Stone <chris.stone> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | chris.stone |
Target Milestone: | --- | Keywords: | Reopened |
Target Release: | --- | Flags: | chris.stone:
fedora-review+
|
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
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: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 163779 |
Description
Hans de Goede
2007-02-01 09:10:56 UTC
The music issue has been cleared, I got the following answer from the author of the music: "Hi! 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. Standard review comment for Hans' packages: icon-theme?? ;-) 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. 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. ==== REVIEW CHECKLIST ==== - 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 44023431a37932ab44a9cf0be9230714 Duel3_20060225_src.zip 0aa9fa786257d6f1d6dd79d2bf591070 Duel3_20060225_bin.zip - 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 (In reply to comment #5) > ==== MUST FIX ==== > - Requires icon cache package to pull in icon dir Done > - 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: http://people.atrpms.net/~hdegoede/duel3.spec SRPM URL: http://people.atrpms.net/~hdegoede/duel3-0.1-0.2.20060225.fc7.src.rpm Hans, please in the future follow the new guidelines outlined here: http://fedoraproject.org/wiki/WarrenTogami/ReviewWithFlags 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. APPROVED Package pre-import admin stuff done. Feel free to import. Imported and build, closing. (Just fixing...) |