Bug 671095 - Review Request: rubygem-pr_geohash - GeoHash encode/decode library for pure Ruby
Summary: Review Request: rubygem-pr_geohash - GeoHash encode/decode library for pure Ruby
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Michal Fojtik
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-01-20 10:09 UTC by Vít Ondruch
Modified: 2011-02-04 11:01 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-02-04 11:01:30 UTC
Type: ---
mfojtik: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Vít Ondruch 2011-01-20 10:09:01 UTC
Spec URL: http://people.redhat.com/vondruch/rubygem-pr_geohash.spec
SRPM URL: http://people.redhat.com/vondruch/rubygem-pr_geohash-1.0.0-1.fc14.src.rpm

Description: GeoHash encode/decode library for pure Ruby.
It's implementation of http://en.wikipedia.org/wiki/Geohash

Koji scratch build for rawhide:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2732801

Comment 1 Michal Fojtik 2011-02-02 14:19:20 UTC
Taking this package for an review.

Comment 2 Michal Fojtik 2011-02-02 14:27:38 UTC
Some notes:

> Requires: rubygems >= 0
> Requires: ruby >= 0
> BuildRequires: rubygems >= 0
> BuildRequires: ruby >= 0

'>= 0' is not necessary here, you can safely omit that

Formal review:

    Packages that contain Ruby Gems must be called rubygem-%{gemname} where gemname is the name from the Gem's specification.

OK

    The Source of the package must be the full URL to the released Gem archive; the version of the package must be the Gem's version

OK

    The package must have a Requires and a BuildRequires on rubygems

OK

    The package must provide rubygem(%{gemname}) where gemname is the name from the Gem's specification. 

OK

    For every dependency on a Gem named gemdep, the package must contain a   Requires on rubygem(%{gemdep}) with the same version constraints as the Gem

OK

    The %prep and %build sections of the specfile should be empty.

OK

    The Gem must be installed into %{gemdir} defined as 

%global gemdir %(ruby -rubygems -e 'puts Gem::dir' 2>/dev/null)

OK

    The package must own the following files and directories: 

%{gemdir}/gems/%{gemname}-%{version}/
%{gemdir}/cache/%{gemname}-%{version}.gem
%{gemdir}/specifications/%{gemname}-%{version}.gemspec


OK

    Architecture-specific content must not be installed into %{gemdir}

OK (there is no architecture content)

    If the Gem only contains pure Ruby code, it must be marked as BuildArch: noarch. 

OK


All other things looks good to me, so REVIEW+. Please fix that version markers before you import this package to GIT.

Comment 3 Vít Ondruch 2011-02-02 16:36:42 UTC
Thank you for your review.

Comment 4 Vít Ondruch 2011-02-02 16:38:22 UTC
New Package SCM Request
=======================
Package Name: rubygem-pr_geohash
Short Description: GeoHash encode/decode library for pure Ruby
Owners: vondruch
Branches: 
InitialCC:

Comment 5 Kevin Fenzi 2011-02-03 21:14:18 UTC
Git done (by process-git-requests).


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