[Bug 820970] Review Request: perl-Task-Moose - install Moose and Moose extensions

2012-05-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=820970

--- Comment #1 from Petr Pisar  2012-05-14 07:11:14 EDT ---
Source tar ball is original. Ok.
URL and Source0 are usable. Ok.

FIX: Title of this bug report does not match summary of the package.

Summary verified from lib/Task/Moose.pm.
License verified from lib/Task/Moose.pm.

FIX: The description refers to a list of module which is not listed. List them
too or remove the confusing sentence.
FIX: The description states it installs something _optionally_. RPM does not
support optional dependencies. You need yo make all listed modules
(build-)requires.

No XS code, noarch BuildArch is Ok.

TODO: Constrain perl(inc::Module::Install) with >= 0.75 version. EPEL still
provides older version.

FIX: Package does not build. The generated Makefile is interactive.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 820970] Review Request: perl-Task-Moose - install Moose and Moose extensions

2012-05-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=820970

Petr Pisar  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||ppi...@redhat.com
 AssignedTo|nob...@fedoraproject.org|ppi...@redhat.com
   Flag||fedora-review?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review