Bug 711048 - Review Request: naev-data - Data files for NAEV game
Summary: Review Request: naev-data - Data files for NAEV game
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 711047
TreeView+ depends on / blocked
 
Reported: 2011-06-06 11:27 UTC by Jonathan Dieter
Modified: 2011-07-08 18:07 UTC (History)
3 users (show)

Fixed In Version: naev-data-0.5.0-4.fc15
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-07-08 18:07:18 UTC
Type: ---
Embargoed:
tcallawa: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Jonathan Dieter 2011-06-06 11:27:14 UTC
Spec URL:
http://www.lesloueizeh.com/jdieter/naev-data.spec
SRPM URL:
http://www.lesloueizeh.com/jdieter/naev-data-0.5.0-1.src.rpm
NoArch F15 RPM:
http://www.lesloueizeh.com/jdieter/naev-data-0.5.0-1.noarch.rpm

NAEV is a 2D space trading and combat game, in a similar vein to Escape
Velocity.  This package contains the data files for the game.

Comment 1 Jonathan Dieter 2011-06-06 11:30:28 UTC
The game package is at #711047

Comment 2 Jonathan Dieter 2011-06-27 15:40:54 UTC
Did some cleanup on the spec file as per comments in bug #711047.

Spec URL:
http://www.lesloueizeh.com/jdieter/naev-data.spec

Comment 3 Tom "spot" Callaway 2011-06-28 16:13:34 UTC
Two things:

* You don't need the %defattr line anymore. I probably should have called it out on the other review too, but its really a minor item.

* The proper license for this data is: GPLv3 or GPLv3+ or GPLv2+ or Public Domain or CC-BY or CC-BY-SA

(Yes, I looked it up when I did my package, thats how I know. :) )

Make those two changes and I'll approve this.

Comment 4 Jonathan Dieter 2011-06-28 18:32:24 UTC
I've removed the defattr line and fixed the license.  Thanks for looking that up!

Spec URL:
http://www.lesloueizeh.com/jdieter/naev-data.spec

Comment 5 Tom "spot" Callaway 2011-06-28 18:59:47 UTC
APPROVED.

Comment 6 Jonathan Dieter 2011-06-29 06:22:59 UTC
New Package SCM Request
=======================
Package Name: naev-data
Short Description: Data files for NAEV
Owners: jdieter
Branches: f14 f15 el6
InitialCC:

Comment 7 Gwyn Ciesla 2011-06-29 12:00:45 UTC
Git done (by process-git-requests).

Comment 8 Fedora Update System 2011-06-29 17:38:18 UTC
naev-data-0.5.0-4.fc15,naev-0.5.0-3.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/naev-data-0.5.0-4.fc15,naev-0.5.0-3.fc15

Comment 9 Fedora Update System 2011-06-29 21:54:19 UTC
naev-data-0.5.0-4.fc15, naev-0.5.0-3.fc15 has been pushed to the Fedora 15 testing repository.

Comment 10 Fedora Update System 2011-07-08 18:07:08 UTC
naev-data-0.5.0-4.fc15, naev-0.5.0-3.fc15 has been pushed to the Fedora 15 stable repository.


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