Bug 1228238 - Review Request: perl-VM-EC2 - Control the Amazon EC2 and Eucalyptus Clouds
Summary: Review Request: perl-VM-EC2 - Control the Amazon EC2 and Eucalyptus Clouds
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Petr Šabata
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1228217
Blocks: 1228241
TreeView+ depends on / blocked
 
Reported: 2015-06-04 13:03 UTC by Petr Pisar
Modified: 2015-06-19 13:29 UTC (History)
2 users (show)

Fixed In Version: perl-VM-EC2-1.28-1.fc23
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-06-19 13:29:19 UTC
Type: ---
Embargoed:
psabata: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
CPAN 104957 0 None None None Never
CPAN 104961 0 None None None Never

Description Petr Pisar 2015-06-04 13:03:06 UTC
Spec URL: https://ppisar.fedorapeople.org/perl-VM-EC2/perl-VM-EC2.spec
SRPM URL: https://ppisar.fedorapeople.org/perl-VM-EC2/perl-VM-EC2-1.28-1.fc23.src.rpm
Description:
This is an interface to the 2013-07-15 version of the Amazon AWS API
<http://aws.amazon.com/ec2>. This library will also support the Eucalyptus
open source cloud <http://open.eucalyptus.com>.

Fedora Account System Username: ppisar

Comment 1 Petr Šabata 2015-06-18 14:48:16 UTC
The patch makes sense.

I suppose the License tag is fine for now.  We'll see what upstream says.  It can be ammended later.

You can drop some build-time dependencies:
- File::Find and Getopt::Long -- bin/* aren't tested at all
- LWP::UserAgent and Storable -- these modules aren't tested either


No blockers, approving.

Comment 2 Petr Pisar 2015-06-19 05:40:03 UTC
New Package SCM Request
=======================
Package Name: perl-VM-EC2
Short Description: Control the Amazon EC2 and Eucalyptus Clouds
Upstream URL: http://search.cpan.org/dist/VM-EC2/
Owners: ppisar jplesnik psabata
Branches: 
InitialCC: perl-sig

Comment 3 Gwyn Ciesla 2015-06-19 12:24:36 UTC
Git done (by process-git-requests).

Comment 4 Petr Pisar 2015-06-19 13:29:19 UTC
Thank you for the review and the repository.

I removed the unneeded build-time dependencies.


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