Bug 714327 (libtpcimgio) - Review Request: libtpcimgio - Turku PET Centre for image file input and output procedures
Summary: Review Request: libtpcimgio - Turku PET Centre for image file input and outpu...
Keywords:
Status: CLOSED ERRATA
Alias: libtpcimgio
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Richard Shaw
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 714326
Blocks: xmedcon
TreeView+ depends on / blocked
 
Reported: 2011-06-18 04:16 UTC by Ankur Sinha (FranciscoD)
Modified: 2011-08-22 15:16 UTC (History)
3 users (show)

Fixed In Version: libtpcimgio-1.5.10-4.fc16
Clone Of:
Environment:
Last Closed: 2011-08-07 08:49:58 UTC
Type: ---
Embargoed:
hobbes1069: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Ankur Sinha (FranciscoD) 2011-06-18 04:16:13 UTC
Spec URL: http://ankursinha.fedorapeople.org/libtpcimgio/libtpcimgio.spec
SRPM URL: http://ankursinha.fedorapeople.org/libtpcimgio/libtpcimgio-1.5.10-1.fc16.src.rpm

Description: 
The libtpcimgio library is a collection of commonly used C files 
in Turku PET Centre for image file input and output procedures. 
Libtpcimgio library supports Analyze 7.5, Ecat 6.x, Ecat 7.x and 
partly interfile formats.

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

[ankur@ankur ~]$ rpmlint /tmp/mock-repo/libtpcimgio-*  ~/rpmbuild/SPECS/libtpcimgio.spec 
libtpcimgio.i686: W: spelling-error Summary(en_US) Turku -> Turk, Turks, Turk u
libtpcimgio.i686: W: no-documentation
libtpcimgio.i686: W: no-manual-page-for-binary libtpcimgio
libtpcimgio.src: W: spelling-error Summary(en_US) Turku -> Turk, Turks, Turk u
libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/sif.c
libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/img_ana.c
libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/img_e63.c
libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/ecat7r.c
libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/analyze.c
libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/ecat63p.c
libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/img.c
libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/libtpcimgiov.c
libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/imgunit.c
libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/ecat63w.c
libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/img_e7.c
libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/sifio.c
libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/ecat7w.c
libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/imgmax.c
libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/ecat7ml.c
libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/imgdecay.c
libtpcimgio-devel.i686: W: no-documentation
4 packages and 1 specfiles checked; 16 errors, 5 warnings.

Comment 1 Richard Shaw 2011-07-28 13:59:53 UTC
Ok, pretty much the same here as:

https://bugzilla.redhat.com/show_bug.cgi?id=714326#c2

So I wont repeat myself.

SPEC: http://hobbes1069.fedorapeople.org/libtpcimgio.spec

Go ahead a make a new SRPM for both packages and I'll start going through the guideline checklist.

Comment 3 Richard Shaw 2011-08-01 15:21:13 UTC
Sorry, got busy but I'm back on the review!

Richard

Comment 4 Richard Shaw 2011-08-01 15:35:42 UTC
Ok, same thing with the BR's for graphviz, other than that I think we're good.

Richard

Comment 6 Richard Shaw 2011-08-05 14:26:12 UTC
+: OK
-: must be fixed
=: should be fixed (at your discretion)
?: Question or clairification needed
N: not applicable

MUST:
[+] rpmlint output: shown in comment: No unexpected output
[+] follows package naming guidelines
[+] spec file base name matches package name
[+] package meets the packaging guidelines
[+] package uses a Fedora approved license: LGPLv2+
[+] license field matches the actual license.
[N] license file is included in %doc:
[+] spec file is in American English
[+] spec file is legible
[+] sources match upstream: md5sums match: ced548015aae0c1f50827b6d51aade9c
[+] package builds on at least one primary arch: Tested F14/F15
[N] appropriate use of ExcludeArch
[+] all build requirements in BuildRequires
[N] spec file handles locales properly
[N] ldconfig in %post and %postun
[+] no bundled copies of system libraries
[N] no relocatable packages
[N] package owns all directories that it creates
[+] no files listed twice in %files
[+] proper permissions on files
[+] consistent use of macros
[+] code or permissible content
[N] large documentation in -doc
[+] no runtime dependencies in %doc
[+] header files in -devel
[+] static libraries in -static: In -devel with -static Provides
[+] .so in -devel
[+] -devel requires main package
[+] package contains no libtool archives
[N] package contains a desktop file, uses desktop-file-install/validate
[+] package does not own files/dirs owned by other packages
[+] all filenames in UTF-8

SHOULD:
[N] query upstream for license text
[N] description and summary contains available translations
[+] package builds in mock
[+] package builds on all supported arches
[?] package functions as described: Did not test.
[+] sane scriptlets
[N] subpackages require the main package (other than -devel)
[N] placement of pkgconfig files
[N] file dependencies versus package dependencies
[=] package contains man pages for binaries/scripts:

*** APPROVED ***

Comment 7 Ankur Sinha (FranciscoD) 2011-08-05 14:32:59 UTC
Thank you for the review Richard :D

New Package SCM Request
=======================
Package Name: libtpcimgio
Short Description: Turku PET Centre for image file input and output procedures
Owners: ankursinha
Branches: f14 f15 f16
InitialCC:

Comment 8 Gwyn Ciesla 2011-08-05 14:46:54 UTC
Git done (by process-git-requests).

Comment 9 Fedora Update System 2011-08-06 07:36:08 UTC
libtpcimgio-1.5.10-3.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/libtpcimgio-1.5.10-3.fc15

Comment 10 Fedora Update System 2011-08-06 07:36:16 UTC
libtpcimgio-1.5.10-3.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/libtpcimgio-1.5.10-3.fc16

Comment 11 Fedora Update System 2011-08-06 07:36:24 UTC
libtpcimgio-1.5.10-3.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/libtpcimgio-1.5.10-3.fc14

Comment 12 Ankur Sinha (FranciscoD) 2011-08-07 08:49:58 UTC
Built and pushed to repos. Closing.

Comment 13 Fedora Update System 2011-08-08 19:19:54 UTC
libtpcimgio-1.5.10-4.fc14, libtpcmisc-1.4.8-4.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/libtpcimgio-1.5.10-4.fc14,libtpcmisc-1.4.8-4.fc14

Comment 14 Fedora Update System 2011-08-08 19:22:05 UTC
libtpcimgio-1.5.10-4.fc16, libtpcmisc-1.4.8-4.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/libtpcimgio-1.5.10-4.fc16,libtpcmisc-1.4.8-4.fc16

Comment 15 Fedora Update System 2011-08-08 19:22:35 UTC
libtpcimgio-1.5.10-4.fc15, libtpcmisc-1.4.8-4.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/libtpcimgio-1.5.10-4.fc15,libtpcmisc-1.4.8-4.fc15

Comment 16 Fedora Update System 2011-08-17 00:56:24 UTC
libtpcimgio-1.5.10-4.fc15, libtpcmisc-1.4.8-4.fc15 has been pushed to the Fedora 15 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 17 Fedora Update System 2011-08-17 01:10:29 UTC
libtpcimgio-1.5.10-4.fc14, libtpcmisc-1.4.8-4.fc14 has been pushed to the Fedora 14 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 18 Fedora Update System 2011-08-22 15:16:47 UTC
libtpcimgio-1.5.10-4.fc16, libtpcmisc-1.4.8-4.fc16 has been pushed to the Fedora 16 stable repository.  If problems still persist, please make note of it in this bug report.


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