[Bug 539268] Review Request: rubygem-will_paginate - Most awesome pagination solution for Rail

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=539268





--- Comment #4 from Darryl L. Pierce <dpierce@xxxxxxxxxx>  2009-12-08 10:51:26 EDT ---
(In reply to comment #1)
> Some quick notes:
> 
> * %define -> %global
>   - Now Fedora prefers to use %global than %define:

Done.

> * License
>   - is MIT

Done.

> * Source
>   - Please use http://gems.rubyforge.org/gems/%{gemname}-%{version}.gem

Done.

> * Requires/BuildRequires
>   - "Requires: ruby(abi) = 1.8" is a must:
>     https://fedoraproject.org/wiki/Packaging/Ruby#Ruby_Packaging_Guidelines
>     Also I usually suggest to add "BuildRequires: ruby(abi) = 1.8"
>     for consistency.
> 
>   - Please check the needed Requires manually.
>     For example (only an example), lib/will_paginate.rb contains:
> -------------------------------------------------------------
>      1  require 'active_support'
> -------------------------------------------------------------
>     This means this binary rpm should have at least
>     "Requires: rubygem(activesupport)".
>     ! Note: use "rubygem(activesupport)" virtual Provides style for
>       Requires instead of using rpm names directly, ref:
>       https://fedoraproject.org/wiki/Packaging/Perl#Perl_Requires_and_Provides

Done. Added a few BuildRequires and Requires for running tests.

> * Using macros
>   - As %geminstdir is defined, please replace
> %{gemdir}/gems/%{gemname}-%{version}/
>     (in %files) with this.

Done.

>   - %ruby_sitelib seems to be used nowhere (in this spec file).

Removed.

> * Duplicate files
>   - Please fix below:
> -------------------------------------------------------------
>     57  warning: File listed twice:
> /usr/lib/ruby/gems/1.8/gems/will_paginate-2.3.11/CHANGELOG.rdoc
>     58  warning: File listed twice:
> /usr/lib/ruby/gems/1.8/gems/will_paginate-2.3.11/LICENSE
>     59  warning: File listed twice:
> /usr/lib/ruby/gems/1.8/gems/will_paginate-2.3.11/README.rdoc
> -------------------------------------------------------------
>     https://fedoraproject.org/wiki/Packaging/Guidelines#Duplicate_Files
>     - Note: please keep these files marked as %doc.

Fixed.

> * Test program
>   - As this rpm contains %geminstdir/test/ please add %check section and
>     execute some test program (some extra BuildRequires are also needed).  

Done.

Updated spec: http://mcpierce.fedorapeople.org/rpms/rubygem-will_paginate.spec
Updated SRPM:
http://mcpierce.fedorapeople.org/rpms/rubygem-will_paginate-2.3.11-2.fc12.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]