[Bug 166506] Review Request: python-basemap - basemap toolkit for matplotlib

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: python-basemap - basemap toolkit for matplotlib


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=166506





------- Additional Comments From orion at cora.nwra.com  2005-11-20 18:38 EST -------
(In reply to comment #8)
> Could you please open another entry for python-basemap-data? 

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=173778

> In the patch that you apply to python-basemap I see this: 
>  
> +# Always build pyshapelib for RPM consistency, uses system shapelib 
>  
> Why? 
>  
> After this will be build in a safe environment so no problem, OTHO maybe it 
> could be interesting to package pyshapelib by itself, no? 
>  
> Is this the only place where it is used? 
> I saw a message from Jeff in the matplotlib list 
> where he says that he rewrote some part that required one of the versions 
> of array (either Numeric or num) in python thus creating pyshapelib. 

Two issues here:

- If I build the package on my system and python-basemap is already installed it
won't build the pyshapelib extensions resulting in failure or a bad
(inconsistent) rpm.  Since I don't always build in mock, I patched setup.py to
always build it.

- I'm using the system shapelib because I think it is *bad* for packages to
provide their own copies of libraries that are already provided elsewhere.  I
don't think it's worth packaging pyshapelib separately at the moment, but we may
wan't to split if someone else want's it.  Biggest issue is that I'm not sure
there is really an *official* pyshapelib anywhere.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.