Bug 719067 - Review Request: rubygem-hmac - This module provides common interface to HMAC functionality
Summary: Review Request: rubygem-hmac - This module provides common interface to HMAC ...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Mo Morsi
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 719073
TreeView+ depends on / blocked
 
Reported: 2011-07-05 16:32 UTC by Chris Lalancette
Modified: 2011-07-20 17:30 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-07-20 17:30:05 UTC
Type: ---
Embargoed:
mmorsi: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Chris Lalancette 2011-07-05 16:32:07 UTC
Spec URL: http://people.redhat.com/clalance/rubygem-hmac/rubygem-hmac.spec
SRPM URL: http://people.redhat.com/clalance/rubygem-hmac/rubygem-hmac-0.4.0-1.fc14.src.rpm

Description:
This module provides common interface to HMAC functionality. HMAC is a kind of
"Message Authentication Code" (MAC) algorithm whose standard is documented in
RFC2104. Namely, a MAC provides a way to check the integrity of information
transmitted over or stored in an unreliable medium, based on a secret key.
Originally written by Daiki Ueno. Converted to a RubyGem by Geoffrey
Grosenbach


[clalance@localhost SPECS]$ rpmlint rubygem-hmac.spec
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
[clalance@localhost SPECS]$ rpmlint ../SRPMS/rubygem-hmac-0.4.0-1.fc14.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

Comment 1 Mo Morsi 2011-07-14 20:13:02 UTC
Taking this one

Comment 2 Mo Morsi 2011-07-14 20:19:33 UTC
Overall looks good, koji build is green:

http://koji.fedoraproject.org/koji/taskinfo?taskID=3199493

A few specific nuances though:

- please replace all the %defines at the top w/ %globals 

http://fedoraproject.org/wiki/Packaging/Guidelines#.25global_preferred_over_.25define


- please enable --ri documentaion on the gem install command

- lib/hmac.rb is licensed under the ruby license and thus the package license must be set to MIT and Ruby accordingly

- please mark the test directory as %doc

- since the test directory is included, it wouldn't be a bad idea to include a %check section to run the suite

- you need a Requires: ruby(abi) = 1.8 and a BuildRequires: ruby as instructed here

http://fedoraproject.org/wiki/Packaging:Ruby#Ruby_Packaging_Guidelines

Comment 3 Chris Lalancette 2011-07-19 17:56:08 UTC
(In reply to comment #2)
> Overall looks good, koji build is green:
> 
> http://koji.fedoraproject.org/koji/taskinfo?taskID=3199493
> 
> A few specific nuances though:
> 
> - please replace all the %defines at the top w/ %globals 
> 
> http://fedoraproject.org/wiki/Packaging/Guidelines#.25global_preferred_over_.25define

Oops, fixed.

> 
> 
> - please enable --ri documentaion on the gem install command

Done, but the use of --ri is very inconsistent in Fedora.  Many packages don't have it, but some do.

> 
> - lib/hmac.rb is licensed under the ruby license and thus the package license
> must be set to MIT and Ruby accordingly

Fixed.

> 
> - please mark the test directory as %doc

Fixed.

> 
> - since the test directory is included, it wouldn't be a bad idea to include a
> %check section to run the suite

Fixed.

> 
> - you need a Requires: ruby(abi) = 1.8 and a BuildRequires: ruby as instructed
> here
> 
> http://fedoraproject.org/wiki/Packaging:Ruby#Ruby_Packaging_Guidelines

Done.

New SRPM: http://people.redhat.com/clalance/rubygem-hmac/rubygem-hmac-0.4.0-2.fc14.src.rpm
New SPEC: http://people.redhat.com/clalance/rubygem-hmac/rubygem-hmac.spec

Comment 4 Mo Morsi 2011-07-20 13:26:46 UTC
Looks good. APPROVED.

Comment 5 Chris Lalancette 2011-07-20 13:52:46 UTC
New Package SCM Request
=======================
Package Name: rubygem-hmac
Short Description: This module provides common interface to HMAC functionality
Owners: clalance mmorsi

Comment 6 Gwyn Ciesla 2011-07-20 15:46:28 UTC
Git done (by process-git-requests).


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