Bug 173778

Summary: Review Request: python-basemap-data - Data for python-basemap
Product: [Fedora] Fedora Reporter: Orion Poplawski <orion>
Component: Package ReviewAssignee: José Matos <jamatos>
Status: CLOSED NEXTRELEASE QA Contact: David Lawrence <dkl>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: 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: ---
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!