Bug 639286

Summary: Review Request: rubygem-amazon-ec2 - A Ruby library for accessing the Amazon Web Services
Product: [Fedora] Fedora Reporter: Michal Fojtik <mfojtik>
Component: Package ReviewAssignee: Mamoru TASAKA <mtasaka>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, notting
Target Milestone: ---Flags: mtasaka: fedora-review+
kevin: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: rubygem-amazon-ec2-0.9.15-3.fc14 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-10-11 19:24:20 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

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.