Bug 257741
| Summary: | Review Request: g2clib - GRIB2 encoder/decoder and search/indexing routines in C | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Patrice Dumas <pertusus> |
| Component: | Package Review | Assignee: | Orion Poplawski <orion> |
| Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | fedora-package-review, notting, tcallawa |
| Target Milestone: | --- | Flags: | orion:
fedora-review+
kevin: fedora-cvs+ |
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2007-12-14 23:55:57 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: | 381241 | ||
|
Description
Patrice Dumas
2007-08-27 19:36:35 UTC
Good: - rpmlint checks return nothing - package meets naming guidelines - package meets packaging guidelines - spec file legible, in am. english - source matches upstream - package compiles on devel (x86) - no missing BR - no unnecessary BR - no locales - not relocatable - owns all directories that it creates - no duplicate files - permissions ok - %clean ok - macro use consistent - code, not content - no need for -docs - nothing in %doc affects runtime - no need for .desktop file - devel package ok - no .la files I went ahead and updated to 1.0.5: http://www.cora.nwra.com/~orion/fedora/g2clib-1.0.5-1.fc8.src.rpm Question: License is listed as Public Domain. I agree that this is probably the case as it is US Gov. work, but there seems to be absolutely no mention of this anywhere in the source. Not sure if this is necessary for public domain. CC'ing Spot for clarification. Does upstream has any interest in supporting building shared libraries? Otherwise, everything looks good. I have asked clarification about the license and they confirmed that it is public domain. I don't know about shared libraries, but upstream is not easily contacted, there is a person who answers mails and could answer the license issue, but for technical issues it is passed to other people and they never responded to my request to rename the fortran lib to avoid name clashes. But you can contact them if you want. It's probably Public Domain, especially given that it is a US Government created work. Nevertheless, if you've got that clarification where upstream confirms that it is public domain, include a copy of that email as a text file in the package. Here it is with the message http://www.lmd.jussieu.fr/~pdlmd/g2clib-1.0.5-2.fc9.src.rpm http://www.lmd.jussieu.fr/~pdlmd/g2clib.spec Looks good to me. Approved. Please add me as a co-owner so I can keep track of changes (so I can rebuild ncl as needed). New Package CVS Request ======================= Package Name: g2clib Short Description: GRIB2 encoder/decoder and search/indexing routines in C Owners: pertusus, orion Branches: EL-4 EL-5 F-7 F-8 InitialCC: Cvsextras Commits: Yes cvs done. Built in all branches, thanks. g2clib-1.0.5-2.fc7 has been pushed to the Fedora 7 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update g2clib' g2clib-1.0.5-2.fc8 has been pushed to the Fedora 8 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update g2clib' g2clib-1.0.5-2.fc7 has been pushed to the Fedora 7 stable repository. If problems still persist, please make note of it in this bug report. g2clib-1.0.5-2.fc8 has been pushed to the Fedora 8 stable repository. If problems still persist, please make note of it in this bug report. |