Bug 711230 - Review Request: ruby-rhubarb - simple versioned object-graph persistence for ruby
Summary: Review Request: ruby-rhubarb - simple versioned object-graph persistence for ...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Tom "spot" Callaway
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-06-06 21:55 UTC by Will Benton
Modified: 2012-01-15 20:09 UTC (History)
5 users (show)

Fixed In Version: ruby-rhubarb-0.4.1-1.fc15
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-01-14 04:00:53 UTC
Type: ---
Embargoed:
tcallawa: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Will Benton 2011-06-06 21:55:43 UTC
Spec URL: http://packages.getwallaby.com/ruby-rhubarb.spec
SRPM URL: http://packages.getwallaby.com/ruby-rhubarb-0.3.0-3.fc14.src.rpm
Description: Rhubarb is a simple object-graph persistence framework for Ruby.  It works with the SPQR object-management framework and exposes some useful functionality for versioned data.

Comment 1 Will Benton 2011-09-22 02:37:52 UTC
A new SRPM is available at http://packages.getwallaby.com/ruby-rhubarb-0.4.0-3.fc14.src.rpm

Comment 2 Will Benton 2011-10-31 19:46:11 UTC
The most recent SRPM is available from packages.getwallaby.com.

Comment 3 Tom "spot" Callaway 2011-11-04 18:18:04 UTC
== Review ==

Good:

- rpmlint checks return:
ruby-rhubarb.noarch: W: spelling-error Summary(en_US) versioned -> version ed, version-ed, version
ruby-rhubarb.src: W: spelling-error Summary(en_US) versioned -> version ed, version-ed, version
ruby-rhubarb.src:21: W: unversioned-explicit-provides ruby(rhubarb/rhubarb)

All safe to ignore, although, I would strongly recommend that you consider appending = %{version} to that explicit provides, even if you're not planning on checking version of it at this time.

- package meets naming guidelines
- package meets packaging guidelines
- license (ASL 2.0) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream (288cd1251a41d8daa4dcc081f2f8b65b86eaab246babc3157e0d64eeab552c59)
- package compiles on f16 (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file 

Looks good, so APPROVED. Consider versioning that explicit provides before commit (and, like in the previous review, dropping the Requires: ruby).

Comment 4 Vít Ondruch 2011-11-08 17:58:41 UTC
Hi, could you please enlighten me why did you packaged the library as Ruby library instead of RubyGem?

Comment 5 Will Benton 2011-12-22 16:21:02 UTC
Vit, it is packaged that way so it will work on EL5 (which doesn't have RubyGems).

Comment 6 Will Benton 2011-12-22 16:22:17 UTC
New Package SCM Request
=======================
Package Name: ruby-rhubarb
Short Description: simple object-graph persistence library for Ruby
Owners: willb
Branches: f15 f16
InitialCC:

Comment 7 Gwyn Ciesla 2011-12-22 16:39:35 UTC
Git done (by process-git-requests).

Comment 8 Vít Ondruch 2011-12-23 09:10:04 UTC
(In reply to comment #5)
> Vit, it is packaged that way so it will work on EL5 (which doesn't have
> RubyGems).

Actually this is not true. RubyGems are available in EPEL5. Moreover, you are requesting just Fedora branches, so this argument is irrelevant for Fedora review.

Comment 9 Fedora Update System 2012-01-04 22:19:53 UTC
ruby-rhubarb-0.4.1-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/ruby-rhubarb-0.4.1-1.fc16

Comment 10 Fedora Update System 2012-01-04 22:21:55 UTC
ruby-rhubarb-0.4.1-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/ruby-rhubarb-0.4.1-1.fc15

Comment 11 Fedora Update System 2012-01-05 21:01:22 UTC
Package ruby-rhubarb-0.4.1-1.fc16:
* should fix your issue,
* was pushed to the Fedora 16 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing ruby-rhubarb-0.4.1-1.fc16'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-0121/ruby-rhubarb-0.4.1-1.fc16
then log in and leave karma (feedback).

Comment 12 Fedora Update System 2012-01-14 04:00:53 UTC
ruby-rhubarb-0.4.1-1.fc15 has been pushed to the Fedora 15 stable repository.

Comment 13 Fedora Update System 2012-01-15 20:02:44 UTC
ruby-rhubarb-0.4.1-1.fc16 has been pushed to the Fedora 16 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 14 Fedora Update System 2012-01-15 20:09:02 UTC
ruby-rhubarb-0.4.1-1.fc15 has been pushed to the Fedora 15 stable repository.


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