Bug 173778

Summary:	Review Request: python-basemap-data - Data for python-basemap
Product:	[Fedora] Fedora	Reporter:	Orion Poplawski <orion>
Component:	Package Review	Assignee:	José Matos <jamatos>
Status:	CLOSED NEXTRELEASE	QA Contact:	David Lawrence <dkl>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	fedora-extras-list
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
URL:	http://www.cora.nwra.com/~orion/fedora/
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2006-01-05 18:11:06 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, 166506

Description Orion Poplawski 2005-11-20 23:31:56 UTC

Spec Name or Url: http://www.cora.nwra.com/~orion/fedora/python-basemap-data.spec
SRPM Name or Url:
http://www.cora.nwra.com/~orion/fedora/python-basemap-data-0.7-1.src.rpm
Description: 

Data for python-basemap



The basemap toolkit for matplotlib requires a large amount of map and other data
for producing its images.  This package provides it.

Comment 1 Orion Poplawski 2006-01-03 21:38:45 UTC

*** Bug 176867 has been marked as a duplicate of this bug. ***

Comment 2 José Matos 2006-01-04 20:08:23 UTC

Package review comming soon.

Comment 3 José Matos 2006-01-05 17:21:45 UTC

Review for release 1: 
* RPM name is OK 
* Source basemap-data-0.7.tar.gz is the same as upstream 
* Builds fine in mock in x86_64 
* rpmlint of python-basemap-data looks OK 
* File list of python-basemap-data looks OK 
* License is OK (GPL) matches the package and is included. 
* BR OK 
* Spec file is readable and it is written in American English 
* Package follows guidelines for python packages 
* package constains mainly data but that data is necessary to run 
python-basemap 
 
Needs work: 
* Specfile should be in the format %{name}.spec 
  (wiki: PackageReviewGuidelines) 
 
You can fix this when importing the package. 
 
APPROVED

Comment 4 Orion Poplawski 2006-01-05 18:10:17 UTC

Renamed spec file.  Checked in and built on devel.

Added to owners.list.

Thanks for the review!