Bug 165955 - Review Request: grads Grid Analysis and Display System
Review Request: grads Grid Analysis and Display System
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Ed Hill
David Lawrence
http://grads.iges.org/grads/grads.html
:
Depends On: 165952
Blocks: FE-ACCEPT
  Show dependency treegraph
 
Reported: 2005-08-15 05:59 EDT by Patrice Dumas
Modified: 2010-07-01 13:29 EDT (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2005-12-21 05:12:02 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
tibbs: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Patrice Dumas 2005-08-15 05:59:56 EDT
SRPM Name or Url: http://www.environnement.ens.fr/docs/fc-srpms/grads-1.9b4-1.src.rpm
Description: 

The Grid Analysis and Display System (GrADS) is an interactive desktop tool
that is used for easy access, manipulation, and visualization of earth science
data. The format of the data may be either binary, GRIB, NetCDF, or HDF-SDS
(Scientific Data Sets). GrADS has been implemented worldwide on a variety of
commonly used operating systems and is freely distributed over the Internet.
Comment 1 Patrice Dumas 2005-08-15 06:02:42 EDT
It seems to me that the b in the version is really part of the version and that
the upstream use ascending ascii letters so it shouldn't be problematic.
Comment 2 Ed Hill 2005-08-15 08:45:10 EDT
Hi Patrice, I'll review this package ASAP.
Comment 3 Ed Hill 2005-08-16 00:10:32 EDT
Hi Patrice, I started to do a review of GrADS and immediately found the
opendap-devel BuildRequires.  Right now, opendap only builds on FC3.  It fails
to build on FC-4 and later (devel) since it has a number of problems with
respect to 64-bit cleanness, etc.  These problems prevent it from building on
x86_64 which then prevents it from building on Fedora Extras.

So, it seems that you have at least three options for GrADS:

 1) leave opendap-devel as a BuildRequires but only target FC-3

 2) remove the opendap-devel BuildRequires (will it still build?)

 3) help Tom Callaway cleanup opendap:
     https://www.redhat.com/archives/fedora-extras-list/2005-August/msg00091.html
     https://www.redhat.com/archives/fedora-extras-list/2005-August/msg00059.html
    so that it builds on FC-4 and later.

I think option #3 is by far the best, but you're free to do as you want.  
And I do hope to help out with the opendap cleanup but it depends on how 
busy (!) I am with work, etc.
Comment 4 Patrice Dumas 2005-08-16 10:01:16 EDT
I can't test a build on 64 bits so I can't help otherwise I would have tried to
help to build on 64 bits.

opendap is optionnal for grads, although it would be better to have it enabled.
I know some people that use grads as their dods client.

I have reposted a grads without opendap support and other changes, especially
those needed to build with the updated libsx.

http://www.environnement.ens.fr/docs/fc-srpms/grads-1.9b4-2.src.rpm
Comment 5 Ed Hill 2005-08-28 20:50:54 EDT
Hi Patrice, heres an incomplete review.  Please post an SRPM that builds
and I'll re-review (and probably approve since the fixes are minor).

needswork:
 - build fails with:
     Error: File not found:
       /var/tmp/grads-1.9b4-2-root-edhill/usr/bin/gradsdods
   which can be fixed by removing:
     %{_bindir}/gradsdods
 - lots of warnings about ignored return values from or implicit 
   re-definitions of fread, fwrite, sscanf, strncpy, etc. which 
   can perhaps be ignored but are worrisome
 - after removing "%{_bindir}/gradsdods", the package builds and 
   rpmlint reports:
      W: grads wrong-file-end-of-line-encoding 
         /usr/share/doc/grads-1.9b4/html/reference_card.pdf
      W: grads wrong-file-end-of-line-encoding 
         /usr/share/doc/grads-1.9b4/html/GrADS.css
      W: grads wrong-file-end-of-line-encoding 
         /usr/share/doc/grads-1.9b4/html/reference_card_scl.pdf
   which can probably be ignored

good:
 - source matches upstream
 - dir ownership and permissions look good
 - names and specfile look OK
 - license is included and looks OK
 - code not content
 - docs look OK
 - no need of -devel package
Comment 6 Deji Akingunola 2005-11-17 22:27:24 EST
Hi Patrice. What's keeping you back from moving this package forward? The last
srpm you provided works for me after following Ed's comment on removing
'%{_bindir}/gradsdods' from spec file and also requiring libsx.  
Comment 7 Patrice Dumas 2005-11-18 05:24:49 EST
There are some licencing issues. After I reported them (maybe without diplomacy)
on the gradsusr list, I got some feedbacks and clarifications for some files,
but not all. And the upstream doesn't seems to be very concerned by this issue.
I was hoping that after some time they would step up and clarify things, but it
didn't happen. The issue is that there are some files with licences incompatible
with the GPL which is the licence for most of the code. It is possible that the
copyright holders are ok with distributing their code under a GPL compatible
licence but don't care enough for doing a move.

Another (but less important) reason why I didn't move is that I submitted a
libdap that should integrate easily in fedora extras but Ed or Tom never
finished the reviewing.

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

In case there is a libdap support there is a need to integrate a patch that
appeared on the gradsusr list, but I'll do that if needed.
Comment 8 Ed Hill 2005-11-25 12:05:37 EST
Hi Patrice, now that libdap is in Extras are there any things you intend to 
update or add/remove here?  Please let me know what the status is and, when 
its ready, I'll give it a (hopefully!) last review.
Comment 9 Patrice Dumas 2005-12-12 19:18:07 EST
Here is a new try:

http://www.environnement.ens.fr/perso/dumas/fc-srpms/grads-1.9b4-3.src.rpm

In that build, I have removed the files with GPL incompatible licences. I also
have applied the patch for a newer libnc-dap.

I have bundled the list of the files from src directory that have an
incompatible licence (in the file grads-removed-files-list), and added a shell
script, grads-remove-files, that removes the files. So to to regenerate the
sources, one could do something like:

spectool grads.spec
sh grads-remove-files

Comment 10 Patrice Dumas 2005-12-13 18:31:49 EST
The previous srpm was certainly broken, or not what was intended. This one
should be right (with minor enhancements, removal of the data source as it is in
the tarball).

http://www.environnement.ens.fr/perso/dumas/fc-srpms/grads-1.9b4-4.src.rpm
Comment 11 Ed Hill 2005-12-19 11:55:44 EST
Hi Patrice, I grabbed grads-1.9b4-4.src.rpm and took a closer look:

good:
 + source "matches" upstream in the sense that you've clearly spent a 
   lot of time carefully extracting only the files that have Fedora-
   compatible licenses and included them in a tarball derived from the 
   upstream tarball (and your extraction script for the un-ship-able 
   files was helpfully provided)
 + licenses are a mess but you've obviously put a lot of effort into 
   tracking down the licenses for every file that matters and have 
   correctly removed the ones that can't be in Extras
 + spec is legible and contains no obvious problems
 + builds in mock on FC-4
 + rpmlint reports:
     W: grads wrong-file-end-of-line-encoding \
        /usr/share/doc/grads-1.9b4/html/reference_card.pdf
     W: grads wrong-file-end-of-line-encoding \
        /usr/share/doc/grads-1.9b4/html/GrADS.css
     W: grads wrong-file-end-of-line-encoding  \
        /usr/share/doc/grads-1.9b4/html/reference_card_scl.pdf
   whiche are all ignore-able warnings
 + I installed and ran a few of the binaries and they worked 
   nicely--exactly per the tutorial

Congrats on beating this old software into an Extras package.  It 
doesn't look like the licensing was any fun to sort out.  In fact, 
the upstream maintainers are clearly linking together software with 
incompatible licenses and perhaps don't realize or even care.  Maybe 
they can contact the origianl authors (eg. Arlindo da Silva who is 
still apparently working at NASA Goddard) and get (some?) of the 
licenses changed so that more can be shipped.

In any case, I don't see any remaining blockers so its APPROVED.
Comment 12 Patrice Dumas 2005-12-21 05:12:02 EST
The upstream maintainers are now fully aware and concerned by the licence
issues, so hopefully this should be solved in the future.

It builds in devel.
Comment 13 Deji Akingunola 2010-07-01 09:15:21 EDT
Package Change Request
======================
Package Name: grads
New Branches: EL-6
Owners: deji    


NOTE: I now maintain grads in Fedora, and Patrice (who takes care of the EPEL branches) does not want to maintain it in EL6, so I'm taking it up.
Comment 14 Patrice Dumas 2010-07-01 09:24:09 EDT
In case it matters, I am ok with Deji.
Comment 15 Jason Tibbitts 2010-07-01 13:29:36 EDT
CVS done (by process-cvs-requests.py).

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