Bug 248513 - Review Request: cdogs-data - Data files for the CDogs game
Review Request: cdogs-data - Data files for the CDogs game
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Chris Mohler
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-07-17 03:48 EDT by Hans de Goede
Modified: 2007-11-30 17:12 EST (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-07-27 04:13:35 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
cr33dog: fedora‑review+
wtogami: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Hans de Goede 2007-07-17 03:48:48 EDT
Spec URL: http://people.atrpms.net/~hdegoede/cdogs-data.spec
SRPM URL: http://people.atrpms.net/~hdegoede/cdogs-data-0.4-1.fc8.src.rpm
Description:
Data files for the CDogs game.

---

Reviewers, notice this is the data for cdogs-sdl which review is bug 248511
Comment 1 Chris Mohler 2007-07-22 17:03:23 EDT
- Passes rpmlint
- Builds in mock (i386)
- Quick glance thought the package:
  * nothing in music/ - is this forthcoming or was something left out?
- rpm installs/program works on i386
* wasn't able to connect to the server upstream to md5 compare
- Freeware license OK

I'm not sure if I have the authority to say it passes review - going to check on
the devel list or on the wiki.
Comment 2 Chris Mohler 2007-07-24 19:29:57 EDT
After a long tour of the wiki I'm marking this accepted. The 'Url' link is still
down, but the 'Source0' checks out.
Comment 3 Hans de Goede 2007-07-25 01:50:26 EDT
Thanks for the review!

New Package CVS Request
=======================
Package Name:      cdogs-data
Short Description: Data files for the CDogs game
Owners:            j.w.r.degoede@hhs.nl
Branches:          FC-6 F-7 devel
InitialCC:         <empty>
Comment 4 Hans de Goede 2007-07-26 14:18:09 EDT
Imported and build. Chris, normally this is the moment when the package gets
closed, so next review please let it open, so that the submitter can close the
review when the package has been build.
Comment 5 Hans de Goede 2007-07-27 04:13:35 EDT
Hmm, I thought you had closed this one too, but appearantly not, closing.

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