Bug 714327 (libtpcimgio)

Summary: Review Request: libtpcimgio - Turku PET Centre for image file input and output procedures
Product: [Fedora] Fedora Reporter: Ankur Sinha (FranciscoD) <sanjay.ankur>
Component: Package ReviewAssignee: Richard Shaw <hobbes1069>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, hobbes1069, notting
Target Milestone: ---Flags: hobbes1069: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: libtpcimgio-1.5.10-4.fc16 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-08-07 08:49:58 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: 714326    
Bug Blocks: 714328    

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.