Bug 1228238

Summary: Review Request: perl-VM-EC2 - Control the Amazon EC2 and Eucalyptus Clouds
Product: [Fedora] Fedora Reporter: Petr Pisar <ppisar>
Component: Package ReviewAssignee: Petr Šabata <psabata>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: package-review, psabata
Target Milestone: ---Flags: psabata: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: perl-VM-EC2-1.28-1.fc23 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-06-19 13:29:19 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:
Bug Depends On: 1228217    
Bug Blocks: 1228241    

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.