Bug 714047 (nifticlib) - Review Request: nifticlib - A set of i/o libraries for reading and writing files in the nifti-1 data format
Summary: Review Request: nifticlib - A set of i/o libraries for reading and writing fi...
Keywords:
Status: CLOSED ERRATA
Alias: nifticlib
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Richard W.M. Jones
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: xmedcon
TreeView+ depends on / blocked
 
Reported: 2011-06-17 08:44 UTC by Ankur Sinha (FranciscoD)
Modified: 2011-08-17 01:22 UTC (History)
3 users (show)

Fixed In Version: nifticlib-2.0.0-2.fc15
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-07-26 17:48:03 UTC
Type: ---
Embargoed:
rjones: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Ankur Sinha (FranciscoD) 2011-06-17 08:44:23 UTC
Spec URL: http://ankursinha.fedorapeople.org/nifticlib/nifticlib.spec
SRPM URL: http://ankursinha.fedorapeople.org/nifticlib/nifticlib-2.0.0-1.fc15.src.rpm

Description: 
Nifticlib is a set of C i/o libraries for reading and writing files in
the nifti-1 data format. nifti-1 is a binary file format for storing
medical image data, e.g. magnetic resonance image (MRI) and functional
MRI (fMRI) brain images.

Comment 1 Ankur Sinha (FranciscoD) 2011-06-17 08:46:17 UTC
[ankur@ankur SRPMS]$ rpmlint ../SPECS/nifticlib.spec nifticlib-2.0.0-1.fc15.src.rpm /var/lib/mock/fedora-rawhide-i386/result/*.rpm
../SPECS/nifticlib.spec:53: E: hardcoded-library-path in $RPM_BUILD_ROOT/usr/lib/*
../SPECS/nifticlib.spec:54: E: hardcoded-library-path in $RPM_BUILD_ROOT/usr/lib/
../SPECS/nifticlib.spec: W: invalid-url Source0: http://downloads.sourceforge.net/nifticlib/nifticlib-2.0.0.tar.gz HTTP Error 404: Not Found
nifticlib.src: W: spelling-error Summary(en_US) nifti -> nifty
nifticlib.src: W: spelling-error %description -l en_US nifti -> nifty
nifticlib.src: W: spelling-error %description -l en_US fMRI -> MRI, f MRI, firm
nifticlib.src:53: E: hardcoded-library-path in $RPM_BUILD_ROOT/usr/lib/*
nifticlib.src:54: E: hardcoded-library-path in $RPM_BUILD_ROOT/usr/lib/
nifticlib.src: W: invalid-url Source0: http://downloads.sourceforge.net/nifticlib/nifticlib-2.0.0.tar.gz HTTP Error 404: Not Found
nifticlib.i686: W: spelling-error Summary(en_US) nifti -> nifty
nifticlib.i686: W: spelling-error %description -l en_US nifti -> nifty
nifticlib.i686: W: spelling-error %description -l en_US fMRI -> MRI, f MRI, firm
nifticlib.i686: W: no-manual-page-for-binary nifti_stats
nifticlib.i686: W: no-manual-page-for-binary nifti_tool
nifticlib.i686: W: no-manual-page-for-binary nifti1_test
nifticlib.src: W: spelling-error Summary(en_US) nifti -> nifty
nifticlib.src: W: spelling-error %description -l en_US nifti -> nifty
nifticlib.src: W: spelling-error %description -l en_US fMRI -> MRI, f MRI, firm
nifticlib.src:53: E: hardcoded-library-path in $RPM_BUILD_ROOT/usr/lib/*
nifticlib.src:54: E: hardcoded-library-path in $RPM_BUILD_ROOT/usr/lib/
nifticlib.src: W: invalid-url Source0: http://downloads.sourceforge.net/nifticlib/nifticlib-2.0.0.tar.gz HTTP Error 404: Not Found
nifticlib-devel.i686: W: no-documentation
6 packages and 1 specfiles checked; 6 errors, 16 warnings.


The errors are irrelevant here.

Comment 2 Richard W.M. Jones 2011-07-26 14:20:17 UTC
Package source matches upstream:

$ sha1sum ../nifticlib-2.0.0.tar.gz nifticlib-2.0.0.tar.gz 
3a6187cb09767f97cef997cf492d89ac3db211df  ../nifticlib-2.0.0.tar.gz
3a6187cb09767f97cef997cf492d89ac3db211df  nifticlib-2.0.0.tar.gz

Comment 3 Richard W.M. Jones 2011-07-26 14:25:39 UTC
Source0 URL is WRONG:

WRONG: http://downloads.sourceforge.net/nifticlib/nifticlib-2.0.0.tar.gz
Right: http://downloads.sourceforge.net/niftilib/nifticlib-2.0.0.tar.gz

Try 'wget'-ing both URLs.

Comment 4 Richard W.M. Jones 2011-07-26 14:26:05 UTC
Koji scratch build worked:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3231346

Comment 5 Richard W.M. Jones 2011-07-26 14:27:40 UTC
I agree that the other rpmlint warnings can be ignored,
including the one about /usr/lib which is bogus.

Comment 6 Richard W.M. Jones 2011-07-26 14:34:01 UTC
+ package name satisfies the packaging naming guidelines
+ specfile name matches the package base name
+ package should satisfy packaging guidelines
+ license meets guidelines and is acceptable to Fedora
+ license matches the actual package license
+ %doc includes license file
+ spec file written in American English
+ spec file is legible
+ upstream sources match sources in the srpm
+ package successfully builds on at least one architecture
n/a ExcludeArch bugs filed
+ BuildRequires list all build dependencies
n/a %find_lang instead of %{_datadir}/locale/*
+ binary RPM with shared library files must call ldconfig in %post and %postun
+ does not use Prefix: /usr
+ package owns all directories it creates
+ no duplicate files in %files
+ consistent use of macros
+ package must contain code or permissible content
+ large documentation files should go in -doc subpackage
+ files marked %doc should not affect package
+ header files should be in -devel
n/a static libraries should be in -static
+ packages containing pkgconfig (.pc) files need 'Requires: pkgconfig'
+ libfoo.so must go in -devel
+ -devel must require the fully versioned base
n/a packages should not contain libtool .la files
n/a packages containing GUI apps must include %{name}.desktop file
n/a packages must not own files or directories owned by other packages
+ filenames must be valid UTF-8

Optional:

n/a if there is no license file, packager should query upstream
n/a translations of description and summary for non-English languages, if available
+ reviewer should build the package in mock
? the package should build into binary RPMs on all supported architectures
? review should test the package functions as described
+ scriptlets should be sane
n/a pkgconfig files should go in -devel
+ shouldn't have file dependencies outside /etc /bin /sbin /usr/bin or /usr/sbin

Package is not approved.  Please just fix the Source URL.

Comment 7 Ankur Sinha (FranciscoD) 2011-07-26 15:22:25 UTC
Corrected:

Fresh srpm/spec:

http://ankursinha.fedorapeople.org/nifticlib/nifticlib.spec

http://ankursinha.fedorapeople.org/nifticlib/nifticlib-2.0.0-2.fc15.src.rpm

Thank you for the *quick* review Richard!
Ankur

Comment 8 Richard W.M. Jones 2011-07-26 16:19:48 UTC
Looks good.

-----------------

This package is APPROVED by rjones

-----------------

Comment 9 Ankur Sinha (FranciscoD) 2011-07-26 16:30:55 UTC
Thank you Richard!

Comment 10 Ankur Sinha (FranciscoD) 2011-07-26 16:32:03 UTC
New Package SCM Request
=======================
Package Name: nifticlib
Short Description: A set of i/o libraries for reading and writing files in the nifti-1 data format
Owners: ankursinha
Branches: f14 f15
InitialCC: susmit mrceresa

Comment 11 Gwyn Ciesla 2011-07-26 17:08:45 UTC
Git done (by process-git-requests).

Comment 12 Fedora Update System 2011-07-26 17:45:37 UTC
nifticlib-2.0.0-2.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/nifticlib-2.0.0-2.fc15

Comment 13 Fedora Update System 2011-07-26 17:46:18 UTC
nifticlib-2.0.0-2.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/nifticlib-2.0.0-2.fc14

Comment 14 Ankur Sinha (FranciscoD) 2011-07-26 17:48:03 UTC
Built for rawhide and pushed to f14 f15 for testing. Closing.

Comment 15 Fedora Update System 2011-08-17 00:57:30 UTC
nifticlib-2.0.0-2.fc14 has been pushed to the Fedora 14 stable repository.

Comment 16 Fedora Update System 2011-08-17 01:22:46 UTC
nifticlib-2.0.0-2.fc15 has been pushed to the Fedora 15 stable repository.


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