Bug 491650 - Review Request: libica - Library for accessing ICA hardware crypto on IBM zSeries
Summary: Review Request: libica - Library for accessing ICA hardware crypto on IBM zSe...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Darryl L. Pierce
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-03-23 14:00 UTC by Dan Horák
Modified: 2015-06-22 00:06 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2009-05-06 07:51:03 UTC
Type: ---
Embargoed:
dpierce: fedora-review+
dennis: fedora-cvs+


Attachments (Terms of Use)

Description Dan Horák 2009-03-23 14:00:08 UTC
Spec URL: http://fedora.danny.cz/s390/libica.spec
SRPM URL: http://fedora.danny.cz/s390/libica-2.0-1.el5.src.rpm

Description:
A library of functions and utilities for accessing ICA hardware crypto on IBM zSeries.


koji scratch build: https://s390.koji.fedoraproject.org/koji/taskinfo?taskID=19585

rpmlint is silent

Comment 1 Darryl L. Pierce 2009-04-29 21:09:23 UTC
Is the warning regarding no documentation considered fatal? Other than that, this package looks fine to me. I'll approve this and the documentation issue should be fixed or ignored.


MUST: rpmlint must be run on every package. The output should be posted in the review.[1]

rpmlint output:

(mcpierce@mcpierce-laptop:Desktop)$ for rpm in *.rpm; do rpmlint $rpm; done
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
libica-devel.s390x: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
libica-utils.s390x: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

MUST: The package MUST successfully compile and build into binary rpms on at least one primary architecture. [7]

MUST: Every binary RPM package (or subpackage) which stores shared library files (not just symlinks) in any of the dynamic linker's default paths, must call ldconfig in %post and %postun. [10] [OK]
MUST: If a package contains library files with a suffix (e.g. libfoo.so.1.1), then library files that end in .so (without suffix) must go in a -devel package. [19] [OK]
MUST: In the vast majority of cases, devel packages must require the base package using a fully versioned dependency: Requires: %{name} = %{version}-%{release} [22] [OK]

SHOULD: The reviewer should test that the package builds in mock. [29] [OK]
SHOULD: The package should compile and build into binary rpms on all supported architectures. [30] [OK]

Comment 2 Dan Horák 2009-04-30 08:49:50 UTC
(In reply to comment #1)
> Is the warning regarding no documentation considered fatal? Other than that,
> this package looks fine to me. I'll approve this and the documentation issue
> should be fixed or ignored.

The warning is harmless here, because there is no documentation other than the AUTHORS and LICENSE files and these are included in the main package that will be always installed (it is the library).

New scratch build using the same srpm this time for F-11 is at
https://s390.koji.fedoraproject.org/koji/taskinfo?taskID=55252

Comment 3 Dan Horák 2009-04-30 09:39:40 UTC
New Package CVS Request
=======================
Package Name: libica
Short Description: Library for accessing ICA hardware crypto on IBM zSeries
Owners: sharkcz
Branches: F-11

Comment 4 Dennis Gilmore 2009-05-01 20:51:29 UTC
CVS Done

Comment 5 Dan Horák 2009-05-06 07:51:03 UTC
Imported and built.


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