Bug 639286 - Review Request: rubygem-amazon-ec2 - A Ruby library for accessing the Amazon Web Services
Summary: Review Request: rubygem-amazon-ec2 - A Ruby library for accessing the Amazon ...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-10-01 11:38 UTC by Michal Fojtik
Modified: 2010-10-29 20:38 UTC (History)
2 users (show)

Fixed In Version: rubygem-amazon-ec2-0.9.15-3.fc14
Clone Of:
Environment:
Last Closed: 2010-10-11 19:24:20 UTC
Type: ---
Embargoed:
mtasaka: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Michal Fojtik 2010-10-01 11:38:07 UTC
Spec URL: http://mifo.sk/RPMS/rubygem-amazon-ec2.spec
SRPM URL: http://mifo.sk/RPMS/rubygem-amazon-ec2-0.9.15-1.fc13.src.rpm
Description:

A Ruby library for accessing the Amazon Web Services EC2, ELB, RDS,
Cloudwatch, and Autoscaling APIs.

Comment 1 Michal Fojtik 2010-10-01 11:39:06 UTC
Koji build:

http://koji.fedoraproject.org/koji/taskinfo?taskID=2505945

rpmlint:

$ rpmlint rubygem-amazon-ec2-0.9.15-1.fc13.noarch.rpm 
rubygem-amazon-ec2.noarch: W: no-manual-page-for-binary ec2sh

Comment 2 Mamoru TASAKA 2010-10-05 20:50:40 UTC
Some notes

* Unused macros
  - %ruby_sitelib macro seems used nowhere.

* License tag
  - When the license text says "under the same terms as Ruby",
    this is "GPLv2 or Ruby" on Fedora.

* Explicit version dependencies
  - As far as I checked for these explicit version dependencies.
    it seems only ">= 0.9.8" for rubygem(mocha) is needed.

* (Build)Requires
  - rubygem(mocha) is only for s.add_development_dependency in
    gemspec file and not needed for runtime (Requires).

* Document files
  - Please consider to create -doc subpackage and move document
    files (like test/ Rakefile %{gemdir}/doc/ %geminstdir/foo.gemspec
    and perhaps deps.rip wsdl perftools) into -doc.

Comment 3 Mamoru TASAKA 2010-10-05 20:53:42 UTC
By the way I would appreciate it if you would review my
gem related review request (bug 637904)

Comment 4 Michal Fojtik 2010-10-06 11:08:48 UTC
(In reply to comment #2)
> Some notes
> 
> * Unused macros
>   - %ruby_sitelib macro seems used nowhere.
> 
> * License tag
>   - When the license text says "under the same terms as Ruby",
>     this is "GPLv2 or Ruby" on Fedora.

Fixed.

> 
> * Explicit version dependencies
>   - As far as I checked for these explicit version dependencies.
>     it seems only ">= 0.9.8" for rubygem(mocha) is needed.

Version dependencies removed.

> 
> * (Build)Requires
>   - rubygem(mocha) is only for s.add_development_dependency in
>     gemspec file and not needed for runtime (Requires).

Mocha removed from runtime. Thanks for catching it.

> 
> * Document files
>   - Please consider to create -doc subpackage and move document
>     files (like test/ Rakefile %{gemdir}/doc/ %geminstdir/foo.gemspec
>     and perhaps deps.rip wsdl perftools) into -doc.

Sure, documentation and support files moved to -doc subpackage.

===================================================================

* Wed Oct 06 2010 Michal Fojtik <mfojtik> - 0.9.15-2
- Removed unused macros
- Fixed license and version dependencies
- Moved documentation files into -doc subpackage

SRPM: http://mifo.sk/RPMS/rubygem-amazon-ec2-0.9.15-2.fc13.src.rpm
Spec: http://mifo.sk/RPMS/rubygem-amazon-ec2.spec

Comment 5 Mamoru TASAKA 2010-10-06 17:51:23 UTC
For -2:

* %defattr on -doc subpackage
  - Please set %defattr on -doc subpackage

* Document files
  - license files or so should be in main package. Please
    move the following files to main package, and mark them
    as %doc (in main package)
------------------------------------------------------
%{geminstdir}/ChangeLog
%{geminstdir}/LICENSE
%{geminstdir}/VERSION
%{geminstdir}/README.rdoc
------------------------------------------------------

Please fix the above when importing this package
into Fedora.
------------------------------------------------------
    This package (rubygem-amazon-ec2) is APPROVED
    by mtasaka
------------------------------------------------------

Comment 6 Michal Fojtik 2010-10-07 08:27:53 UTC
Thank you! Both things are fixed now.

New Package SCM Request
=======================
Package Name:      rubygem-amazon-ec2
Short Description:  A Ruby library for accessing the Amazon Web Services
Owners:            mfojtik
Branches:          f12 f13 f14

Comment 7 Kevin Fenzi 2010-10-08 20:33:59 UTC
Git done (by process-git-requests).

Comment 8 Fedora Update System 2010-10-11 08:29:03 UTC
rubygem-amazon-ec2-0.9.15-3.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/rubygem-amazon-ec2-0.9.15-3.fc13

Comment 9 Fedora Update System 2010-10-11 08:34:11 UTC
rubygem-amazon-ec2-0.9.15-3.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/rubygem-amazon-ec2-0.9.15-3.fc14

Comment 10 Fedora Update System 2010-10-11 19:24:15 UTC
rubygem-amazon-ec2-0.9.15-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 11 Fedora Update System 2010-10-29 20:38:18 UTC
rubygem-amazon-ec2-0.9.15-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.