Bug 1242727 - Review Request: perl-MooX-Roles-Pluggable - Moo eXtension for pluggable roles
Summary: Review Request: perl-MooX-Roles-Pluggable - Moo eXtension for pluggable roles
Keywords:
Status: CLOSED ERRATA
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:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-07-14 04:17 UTC by Tim Orling
Modified: 2015-11-01 02:50 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2015-10-16 06:58:18 UTC
Type: ---
Embargoed:
psabata: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Tim Orling 2015-07-14 04:17:13 UTC
Spec URL: https://ttorling.fedorapeople.org/perl-MooX-Roles-Pluggable/perl-MooX-Roles-Pluggable.spec
SRPM URL: https://ttorling.fedorapeople.org/perl-MooX-Roles-Pluggable/perl-MooX-Roles-Pluggable-0.003-1.fc23.src.rpm
Description: This module allows a class consuming several roles based on rules passed to Module::Pluggable::Object.
Fedora Account System Username: ttorling

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

copr:
https://copr.fedoraproject.org/coprs/ttorling/perl-meta-cpan-support/monitor/

This is the fourth of four new packages to support meta-cpan (https://github.com/rehsack/meta-cpan) and Packager::Utils (https://github.com/rehsack/Packager-Utils) for Fedora.

I am in need of a sponsor.

Comment 1 Petr Šabata 2015-07-22 14:31:52 UTC
Based on my feedback from the other two reviews, try to fix this package first.  I would just repeat myself and it will be a good exercise for you :)

Comment 3 Petr Šabata 2015-08-11 14:59:42 UTC
(In reply to Tim Orling from comment #2)
> It appears Module::Pluggable, Moo, Moo::Role are not in EPEL6
> It appears Moo and Moo::Role are not in EPEL7
> 
> Many dependencies for Moo in EPEL7 are not met:
> http://koji.fedoraproject.org/koji/taskinfo?taskID=10613995

I'll consider this Fedora only then.

* Unless you plan to package Moo for EL7, you may use EU::MM 6.76+ NO_PACKLIST feature, available in all stable Fedoras:

  BuildRequires:  perl(ExtUtils::MakeMaker) >= 6.76
  Makefile.PL INSTALLDIRS=vendor NO_PACKLIST=1

  If you do this, you won't need to remove .packlist files in %install.
  This will also allow you to drop the dependencies on coreutils and findutils.

* Missing a build deps -- perl(lib), from t/01-basic.t:10 and t/02-fatal.t:10,
  and perl(Module::Pluggable::Object) from lib/MooX/Roles/Pluggable.pm:7.


That seems to be all.  Good! :)

Comment 5 Petr Šabata 2015-08-13 14:34:43 UTC
Okay, looks fine now.  Approving.

Comment 6 Tim Orling 2015-08-13 22:42:38 UTC
New Package SCM Request
=======================
Package Name: perl-MooX-Roles-Pluggable
Short Description: Moo eXtension for pluggable roles
Upstream URL: http://search.cpan.org/dist/MooX-Roles-Pluggable/
Owners: ttorling psabata
Branches: f21 f22 f23
InitialCC: perl-sig

Comment 7 Gwyn Ciesla 2015-08-17 13:32:13 UTC
Git done (by process-git-requests).

Comment 8 Fedora Update System 2015-08-17 15:11:58 UTC
perl-MooX-Roles-Pluggable-0.003-3.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/perl-MooX-Roles-Pluggable-0.003-3.fc21

Comment 9 Fedora Update System 2015-08-17 15:19:17 UTC
perl-MooX-Roles-Pluggable-0.003-3.fc22 has been submitted as an update for Fedora 22.
https://admin.fedoraproject.org/updates/perl-MooX-Roles-Pluggable-0.003-3.fc22

Comment 10 Fedora Update System 2015-08-17 15:28:01 UTC
perl-MooX-Roles-Pluggable-0.003-3.fc23 has been submitted as an update for Fedora 23.
https://admin.fedoraproject.org/updates/perl-MooX-Roles-Pluggable-0.003-3.fc23

Comment 11 Fedora Update System 2015-10-16 06:58:17 UTC
perl-MooX-Roles-Pluggable-0.003-3.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report.

Comment 12 Fedora Update System 2015-10-16 13:21:07 UTC
perl-MooX-Roles-Pluggable-0.003-3.fc21 has been pushed to the Fedora 21 stable repository. If problems still persist, please make note of it in this bug report.

Comment 13 Fedora Update System 2015-11-01 02:50:02 UTC
perl-MooX-Roles-Pluggable-0.003-3.fc23 has been pushed to the Fedora 23 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.