Bug 717920 - Review Request: rubygem-hike - Find files in a set of paths
Summary: Review Request: rubygem-hike - Find files in a set of paths
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Bohuslav "Slavek" Kabrda
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-06-30 12:25 UTC by Vít Ondruch
Modified: 2012-01-30 14:08 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-01-30 14:08:50 UTC
Type: ---
bkabrda: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Vít Ondruch 2011-06-30 12:25:20 UTC
Spec URL: http://people.redhat.com/vondruch/rubygem-hike.spec
SRPM URL: http://people.redhat.com/vondruch/rubygem-hike-1.1.0-1.fc16.src.rpm
Description: A Ruby library for finding files in a set of paths.

Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3171697

Comment 1 Bohuslav "Slavek" Kabrda 2012-01-30 07:25:14 UTC
I'll take this for a review.

Comment 2 Bohuslav "Slavek" Kabrda 2012-01-30 07:37:19 UTC
- Please update to the latest version of gem (currently 1.2.1).
- The specfile you provided differs from the specfile in SRPM (just the two comments in %check, but please fix this).
- License is MIT, not GPLv2+ or Ruby.
- Please consider updating to the new standard of Rubygems packaging for Rawhide with Ruby 1.9.3 [1] (and please build the package with new Ruby, not the old one - use f17-ruby target in Koji).
- Otherwise the package looks good, but I'd like to see it once more after it gets updated to the latest version, then I'll gladly approve it.

[1] https://fedoraproject.org/wiki/PackagingDrafts/Ruby

Comment 3 Vít Ondruch 2012-01-30 09:34:18 UTC
(In reply to comment #2)
> - Please update to the latest version of gem (currently 1.2.1).

Done

> - The specfile you provided differs from the specfile in SRPM (just the two
> comments in %check, but please fix this).

Done

> - License is MIT, not GPLv2+ or Ruby.

Done

> - Please consider updating to the new standard of Rubygems packaging for
> Rawhide with Ruby 1.9.3 [1] (and please build the package with new Ruby, not
> the old one - use f17-ruby target in Koji).

Done

> - Otherwise the package looks good, but I'd like to see it once more after it
> gets updated to the latest version, then I'll gladly approve it.
> 
> [1] https://fedoraproject.org/wiki/PackagingDrafts/Ruby

Spec URL: http://people.redhat.com/vondruch/rubygem-hike.spec
SRPM URL: http://people.redhat.com/vondruch/rubygem-hike-1.2.1-1.fc17.src.rpm

Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3745471

Comment 4 Bohuslav "Slavek" Kabrda 2012-01-30 09:49:03 UTC
The package looks good now => APPROVED.

Comment 5 Vít Ondruch 2012-01-30 10:36:25 UTC
Thank you for your review!



New Package SCM Request
=======================
Package Name: rubygem-hike
Short Description: Find files in a set of paths
Owners: vondruch
Branches: 
InitialCC:

Comment 6 Gwyn Ciesla 2012-01-30 13:24:25 UTC
Git done (by process-git-requests).


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