Bug 711048

Summary: Review Request: naev-data - Data files for NAEV game
Product: [Fedora] Fedora Reporter: Jonathan Dieter <jonathan>
Component: Package ReviewAssignee: Nobody's working on this, feel free to take it <nobody>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: fedora-package-review, notting, tcallawa
Target Milestone: ---Flags: tcallawa: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: naev-data-0.5.0-4.fc15 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-07-08 18:07:18 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: 711047    

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.