Bug 697779 - Review Request: rubygem-file-tail - File::Tail for Ruby
Summary: Review Request: rubygem-file-tail - File::Tail for 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:44 UTC by Peng Wu
Modified: 2011-04-26 21:53 UTC (History)
5 users (show)

Fixed In Version: rubygem-file-tail-1.0.5-3.fc14
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-04-26 16:25:46 UTC
Type: ---
panemade: fedora-review+
j: fedora-cvs+


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

Comment 1 Parag AN(पराग) 2011-04-19 14:50:11 UTC
Suggestions:
1) Each Ruby package MUST indicate the Ruby ABI version it depends on. Add following at top lines
   %global	rubyabi	1.8
2) missing BR
BuildRequires:	ruby(abi) = %{rubyabi}
3) License looks GPLv2 from README
4) For fedora we don't need buildroot,%clean section and cleaning of buildroot in %install

Comment 2 Parag AN(पराग) 2011-04-19 15:39:45 UTC
you can also add -doc subpackage
%files doc
%defattr(-, root, root, -)
%{geminstdir}/examples
%{geminstdir}/test
%{geminstdir}/Rakefile
%{gemdir}/doc/%{gemname}-%{version}

Comment 3 Parag AN(पराग) 2011-04-20 03:55:51 UTC
oops I just realized that its actually missing Requires and not BR: in above 2)

Comment 5 Vít Ondruch 2011-04-20 10:18:50 UTC
Could you please execute the test suite during build?

Comment 6 Parag AN(पराग) 2011-04-21 06:04:02 UTC
Created attachment 493706 [details]
modification to spec file to follow ruby guidelines

- enable testsuite
- Remove versions from versioned Requires.
- Fix shebang to silent rpmlint warning.
- Fix file permission issues.

Comment 7 Parag AN(पराग) 2011-04-21 06:04:56 UTC
oops enable following line in spec

Requires: rubygem(spruz)

Comment 8 Peng Wu 2011-04-21 06:50:09 UTC
Thanks very much for the help.

Here are the new spec and rpm:
Spec URL: http://pwu.fedorapeople.org/rubygem-file-tail/rubygem-file-tail.spec
SRPM URL:
http://pwu.fedorapeople.org/rubygem-file-tail/rubygem-file-tail-1.0.5-3.fc14.src.rpm

Please review it again.

Comment 9 Parag AN(पराग) 2011-04-21 07:11:17 UTC
Review:

+ koji build -> http://koji.fedoraproject.org/koji/taskinfo?taskID=3015380

+ Upstream source verified as (sha1sum)
2b3bd0f0ae0419f670544064dc27f6286e0fc880  file-tail-1.0.5.gem
2b3bd0f0ae0419f670544064dc27f6286e0fc880  ../SOURCES/file-tail-1.0.5.gem

+ rpmlint gave
rubygem-file-tail.noarch: W: no-manual-page-for-binary rtail
rubygem-file-tail-doc.noarch: W: no-documentation
3 packages and 0 specfiles checked; 0 errors, 2 warnings.

APPROVED.

Comment 10 Peng Wu 2011-04-21 09:13:35 UTC
New Package SCM Request
=======================
Package Name: rubygem-file-tail
Short Description: File::Tail for Ruby
Owners: pwu
Branches: f15 f14 f13 el6
InitialCC: i18n-team petersen jamesni

Comment 11 Jason Tibbitts 2011-04-22 14:27:51 UTC
Git done (by process-git-requests).

Comment 12 Fedora Update System 2011-04-25 05:07:52 UTC
rubygem-file-tail-1.0.5-3.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/rubygem-file-tail-1.0.5-3.fc14

Comment 13 Fedora Update System 2011-04-25 05:08:00 UTC
rubygem-file-tail-1.0.5-3.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/rubygem-file-tail-1.0.5-3.fc13

Comment 14 Fedora Update System 2011-04-25 05:08:08 UTC
rubygem-file-tail-1.0.5-3.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/rubygem-file-tail-1.0.5-3.fc15

Comment 15 Fedora Update System 2011-04-25 20:56:11 UTC
rubygem-file-tail-1.0.5-3.fc13 has been pushed to the Fedora 13 testing repository.

Comment 16 Fedora Update System 2011-04-26 16:25:41 UTC
rubygem-file-tail-1.0.5-3.fc15 has been pushed to the Fedora 15 stable repository.

Comment 17 Fedora Update System 2011-04-26 21:51:42 UTC
rubygem-file-tail-1.0.5-3.fc13 has been pushed to the Fedora 13 stable repository.

Comment 18 Fedora Update System 2011-04-26 21:53:31 UTC
rubygem-file-tail-1.0.5-3.fc14 has been pushed to the Fedora 14 stable repository.


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