Bug 505406 - Review Request: rubygem-rake-compiler - Rake-based Ruby C Extension task generator
Summary: Review Request: rubygem-rake-compiler - Rake-based Ruby C Extension task gene...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Xavier Lamien
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-06-11 19:28 UTC by Mamoru TASAKA
Modified: 2014-09-21 18:43 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2009-07-02 16:39:39 UTC
Type: ---
Embargoed:
lxtnow: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Mamoru TASAKA 2009-06-11 19:28:18 UTC
Spec URL: http://mtasaka.fedorapeople.org/Review_request/rubygem-rake-compiler/rubygem-rake-compiler.spec
SRPM URL: http://mtasaka.fedorapeople.org/Review_request/rubygem-rake-compiler/rubygem-rake-compiler-0.5.0-1.fc.src.rpm

Description: 
rake-compiler aims to help Gem developers while dealing with
Ruby C extensions, simplifiying the code and reducing the duplication.

It follows *convention over configuration* and set an standarized
structure to build and package C extensions in your gems.

This is the result of expriences dealing with several Gems 
that required native extensions across platforms and different 
user configurations where details like portability and 
clarity of code were lacking. 

Koji build
- For F-12: http://koji.fedoraproject.org/koji/taskinfo?taskID=1405896
- For F-11: http://koji.fedoraproject.org/koji/taskinfo?taskID=1405894

Comment 1 Xavier Lamien 2009-07-01 10:42:34 UTC
OK source files match upstream
OK package meets naming and versioning guidelines.
OK specfile is properly named, is cleanly written and uses macros consistently.
OK dist tag is present.
OK Global are used.
OK build root is correct.
OK license field matches the actual license.
OK license is open source-compatible (MIT). License text included in
package in part of reason has been given properly.
OK latest version is being packaged.
OK BuildRequires are proper.
OK %clean is present.
OK package builds in koji (F-11 & Rawhide -- noarch).
OK rpmlint is silent.
OK final provides and requires look sane.
-- %check is present (however, test need more work, not a blocker).
OK owns the directories it creates.
OK doesn't own any directories it shouldn't.
OK no duplicates in %files.
OK file permissions are appropriate.
OK correct scriptlets present.
OK code, not content.
OK huge %docs has been moved to sub-package 
OK package installed properly

According to ruby guideline, the package uses proper ruby specific packaging.


** APPROVED **

ps: sorry for have been so long.

Comment 2 Mamoru TASAKA 2009-07-01 13:16:30 UTC
No problem, thank you for reviewing this ticket.

New Package CVS Request 
=======================
Package Name:       rubygem-rake-compiler
Short Description:  Rake-based Ruby C Extension task generator
Owners:             mtasaka
Branches:           F-11 F-10
InitialCC:          (nobody)

Comment 3 Kevin Fenzi 2009-07-02 05:02:49 UTC
cvs done.

Comment 4 Mamoru TASAKA 2009-07-02 16:39:39 UTC
Rebuilt on all branches, submitted push requests on bodhi, closing.
Thank you for the review and CVS procedure.

Comment 5 Mark Chappell 2010-02-01 16:52:04 UTC
Package Change Request
=======================
Package Name:       rubygem-rake-compiler
Owners:             tremble
Branches:           EL-4 EL-5

Comment 6 Kevin Fenzi 2010-02-01 20:41:41 UTC
cvs done.

Comment 7 Denis Fateyev 2014-09-21 18:38:19 UTC
Current package maintainers have no objections: BZ#1142428

Package Change Request
======================
Package Name: rubygem-rake-compiler
New Branches: epel7
Owners: dfateyev

Comment 8 Gwyn Ciesla 2014-09-21 18:43:32 UTC
Git done (by process-git-requests).


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