Bug 697767 - Review Request: rubygem-spruz - Useful tools library in Ruby
Summary: Review Request: rubygem-spruz - Useful tools library in Ruby
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-04-19 08:01 UTC by James Ni
Modified: 2011-04-26 21:54 UTC (History)
5 users (show)

Fixed In Version: rubygem-spruz-0.2.5-3.fc14
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-04-26 16:22:42 UTC
Type: ---
Embargoed:
panemade: fedora-review+
j: fedora-cvs+


Attachments (Terms of Use)
modified spec to follow ruby guidelines (2.14 KB, text/x-rpm-spec)
2011-04-21 05:58 UTC, Parag AN(पराग)
no flags Details

Description James Ni 2011-04-19 08:01:11 UTC
Spec URL: http://jamesni.fedorapeople.org/rubygem-spruz/rubygem-spruz.spec  
SRPM URL: http://jamesni.fedorapeople.org/rubygem-spruz/rubygem-spruz-0.2.5-1.fc13.src.rpm
Description: 
All the stuff that isn't good/big enough for a real library.

Comment 1 James Ni 2011-04-19 08:25:04 UTC
Koji scratch build
http://koji.fedoraproject.org/koji/taskinfo?taskID=3010159

Comment 2 Parag AN(पराग) 2011-04-21 05:58:37 UTC
Created attachment 493704 [details]
modified spec to follow ruby guidelines

Modifications done are
- Subpackage doc files.
- Drop %%{buildroot}, %%clean section and cleaning 
  of buildroot in %%install.
- Fix the permission issue for enum and install.rb file.
- Fix the shebang in script files.

Comment 3 James Ni 2011-04-21 06:25:13 UTC
Thanks, I have update the package and spec file

Spec URL: http://jamesni.fedorapeople.org/rubygem-spruz/rubygem-spruz.spec  
SRPM URL:
http://jamesni.fedorapeople.org/rubygem-spruz/rubygem-spruz-0.2.5-2.fc13.src.rpm

Comment 4 Parag AN(पराग) 2011-04-21 06:43:29 UTC
Review:
+ Koji build-> http://koji.fedoraproject.org/koji/taskinfo?taskID=3015357
+ upstream source verified as (sha1sum)
55f827c5bff7716c0230465fca9b2d765ca9d612  ../SOURCES/spruz-0.2.5.gem
55f827c5bff7716c0230465fca9b2d765ca9d612  spruz-0.2.5.gem

+ rpmlint gave
rubygem-spruz.noarch: W: no-manual-page-for-binary enum
rubygem-spruz-doc.noarch: W: no-documentation
rubygem-spruz-doc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/spruz-0.2.5/ri/Array/uniq_by%21-i.yaml %21
rubygem-spruz-doc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/spruz-0.2.5/ri/Spruz/Full/full%3f-i.yaml %3f
rubygem-spruz-doc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/spruz-0.2.5/ri/Spruz/Bijection/%5b%5d-c.yaml %5b
rubygem-spruz-doc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/spruz-0.2.5/ri/Spruz/Bijection/%5b%5d-c.yaml %5d
rubygem-spruz-doc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/spruz-0.2.5/ri/Spruz/Minimize/unminimize%21-i.yaml %21
rubygem-spruz-doc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/spruz-0.2.5/ri/Spruz/Bijection/%5b%5d%3d-i.yaml %5b
rubygem-spruz-doc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/spruz-0.2.5/ri/Spruz/Bijection/%5b%5d%3d-i.yaml %5d
rubygem-spruz-doc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/spruz-0.2.5/ri/Spruz/Bijection/%5b%5d%3d-i.yaml %3d
rubygem-spruz-doc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/spruz-0.2.5/ri/Spruz/Minimize/minimize%21-i.yaml %21
rubygem-spruz-doc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/spruz-0.2.5/ri/Spruz/ModuleGroup/%5b%5d-c.yaml %5b
rubygem-spruz-doc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/spruz-0.2.5/ri/Spruz/ModuleGroup/%5b%5d-c.yaml %5d
rubygem-spruz-doc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/spruz-0.2.5/ri/Hash/subhash%21-i.yaml %21
rubygem-spruz-doc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/spruz-0.2.5/ri/Spruz/Shuffle/shuffle%21-i.yaml %21
rubygem-spruz-doc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/spruz-0.2.5/ri/Spruz/HashUnion/%7c-i.yaml %7c
rubygem-spruz-doc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/spruz-0.2.5/ri/Spruz/Generator/%5b%5d-c.yaml %5b
rubygem-spruz-doc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/spruz-0.2.5/ri/Spruz/Generator/%5b%5d-c.yaml %5d
4 packages and 0 specfiles checked; 0 errors, 20 warnings.


I am not able to get testsuite working. 

vondruch,
   Is it possible to get testsuite working here?

Comment 5 Parag AN(पराग) 2011-04-21 09:28:06 UTC
I think rest looks ok. Only need to check if we can have testsuite working here. 
I will not block this review for that.


APPROVED.

Comment 6 James Ni 2011-04-21 10:03:28 UTC
New Package SCM Request
=======================
Package Name: rubygem-spruz
Short Description: All the stuff that isn't good/big enough for a real library.
Owners: jamesni
Branches: f13 f14 f15 el6
InitialCC: pwu petersen i18n-team

Comment 7 Vít Ondruch 2011-04-21 14:01:04 UTC
> I am not able to get testsuite working. 
> 
> vondruch,
>    Is it possible to get testsuite working here?

Add BuildRequires: rubygem(test-unit)

And then execute

RUBYOPT="I%{buildroot}%{geminstdir}/lib" testrb2 tests/test_*.rb

The test-unit which is integral part of Ruby is not that powerful.

Comment 8 James Ni 2011-04-22 03:12:03 UTC
Thanks, just update the package and spec file

Spec URL: http://jamesni.fedorapeople.org/rubygem-spruz/rubygem-spruz.spec
SRPM URL:
http://jamesni.fedorapeople.org/rubygem-spruz/rubygem-spruz-0.2.5-3.fc13.src.rpm

Comment 9 Jason Tibbitts 2011-04-22 14:26:37 UTC
Git done (by process-git-requests).

Comment 10 Fedora Update System 2011-04-25 06:08:23 UTC
rubygem-spruz-0.2.5-3.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/rubygem-spruz-0.2.5-3.fc14

Comment 11 Fedora Update System 2011-04-25 06:08:31 UTC
rubygem-spruz-0.2.5-3.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/rubygem-spruz-0.2.5-3.fc15

Comment 12 Fedora Update System 2011-04-25 06:08:39 UTC
rubygem-spruz-0.2.5-3.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/rubygem-spruz-0.2.5-3.fc13

Comment 13 Fedora Update System 2011-04-25 20:56:57 UTC
Package rubygem-spruz-0.2.5-3.fc14:
* should fix your issue,
* was pushed to the Fedora 14 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing rubygem-spruz-0.2.5-3.fc14'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/rubygem-spruz-0.2.5-3.fc14
then log in and leave karma (feedback).

Comment 14 Fedora Update System 2011-04-26 16:22:36 UTC
rubygem-spruz-0.2.5-3.fc15 has been pushed to the Fedora 15 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 15 Fedora Update System 2011-04-26 21:50:14 UTC
rubygem-spruz-0.2.5-3.fc13 has been pushed to the Fedora 13 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 16 Fedora Update System 2011-04-26 21:53:55 UTC
rubygem-spruz-0.2.5-3.fc14 has been pushed to the Fedora 14 stable repository.  If problems still persist, please make note of it in this bug report.


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