Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 191998 - Review Request: scrip -- spherical coordinate interpolation and remapping package
Summary: Review Request: scrip -- spherical coordinate interpolation and remapping pac...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Patrice Dumas
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-05-16 18:49 UTC by Ed Hill
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-05-17 19:42:30 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
use variables on make command line to change compilation defaults (777 bytes, patch)
2006-05-17 13:48 UTC, Patrice Dumas
no flags Details | Diff

Description Ed Hill 2006-05-16 18:49:16 UTC
Spec URL: http://mitgcm.org/eh3/fedora_misc/scrip.spec
SRPM URL: http://mitgcm.org/eh3/fedora_misc/scrip-1.4-3.src.rpm
Description: scrip -- spherical coordinate interpolation and remapping package

This package installs a single binary ("scrip") and a pdf-formatted 
user's manual.  The scrip program calculates remapping weights for 
gridded data sets on the surface of a sphere.  It reads netCDF files 
that contain the grid information and writes netcdf files that contain 
the remapping weights.

Comment 1 Patrice Dumas 2006-05-17 13:44:04 UTC
With the patch, the rpm flags are not acknowledged and the include and 
libdir are hardcoded, so I propose the attaced patch instead. 


Comment 2 Patrice Dumas 2006-05-17 13:48:54 UTC
Created attachment 129318 [details]
use variables on make command line to change compilation defaults

Comment 3 Ed Hill 2006-05-17 15:04:00 UTC
Hi Patrice, that certainly is an improvement -- thank you!  And here are 
the updated files:

  http://mitgcm.org/eh3/fedora_misc/scrip-1.4-4.src.rpm
  http://mitgcm.org/eh3/fedora_misc/scrip.spec

Comment 4 Patrice Dumas 2006-05-17 16:39:20 UTC
* rpmlint is silent
* naming is good
* follow packaging guidelines
* licence acceptable and included
* spec legible
* source match upstream
* everything else is fine
* builds in mock

APPROVED

comments:
* the timestamp of the sources is wrong, I find it better to have
the right timestamp. In general I use spectool to achive that (or
maybe wget -N). 
* the executables in grids/ may be compiled and distributed maybe with
scrip_ prefixed, and in that case the README file may also be added, 
renamed for example README.utils.
* the bugs file could be in %docs

Comment 5 Jason Tibbitts 2006-05-17 17:08:20 UTC
Did anyone notice that the license is not BSD as indicated in the spec?  I
believe it is free according to the Open Source Definition, but the text
certainly does not resemble the BSD license text and the derivative works clause:

If SOFTWARE is modified to produce derivative works, such modified SOFTWARE
should be clearly marked, so as not to confuse it with the version available
from Los Alamos National Laboratory.

is not present in the BSD license.

Comment 6 Ed Hill 2006-05-17 17:26:27 UTC
Hi Jason, you're right that its not exactly BSD.  It does appear to be 
sufficiently "open" in that it allows modification, redistribution, 
sale, etc.  So I think the licesnse terms are perfectly acceptable for 
inclusion in FE.  The only question that remains is "what should we call 
the license?"

I'll be happy to change the license name in the spec before it gets built 
and pushed (it was just imported).  What would you folks like to call it?  
Woiuld the "SCRIP License" be OK?

Comment 7 Jason Tibbitts 2006-05-17 19:00:31 UTC
I don't think it corresponds to any existing license, but I agree that it's
certainly free enough.  (Item 4 of the Open Source Definition allows requiring
the marking or renaming of derivative works.)

Are there other packages which use the same license?  If not, "SCRIP License" is
probably fine but you'll have to endure the rpmlint warning.

Comment 8 Ed Hill 2006-05-17 19:42:30 UTC
Hi Patrice and Jason, thank you for the patch and speedy reviews!  I've 
incorporated almost all of your changes and will, as soon as I get some 
more free time, add the SCRIP/grids and namelist files.

The package built successfully on devel and FC-4/5 branches have been 
requested.

Thanks again!


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