Bug 1008473 - Review Request: perl-Class-Tiny - Minimalist class construction
Summary: Review Request: perl-Class-Tiny - Minimalist class construction
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: 1008407 1008809
TreeView+ depends on / blocked
 
Reported: 2013-09-16 12:51 UTC by Jitka Plesnikova
Modified: 2013-09-27 00:49 UTC (History)
3 users (show)

Fixed In Version: perl-Class-Tiny-0.009-1.fc20
Clone Of:
Environment:
Last Closed: 2013-09-27 00:49:10 UTC
Type: ---
Embargoed:
psabata: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Jitka Plesnikova 2013-09-16 12:51:16 UTC
Spec URL: http://jplesnik.fedorapeople.org/perl-Class-Tiny/perl-Class-Tiny.spec
SRPM URL: http://jplesnik.fedorapeople.org/perl-Class-Tiny/perl-Class-Tiny-0.008-1.fc21.src.rpm

Description: 
This module offers a minimalist class construction kit in around 120 lines
of code. Here is a list of features:

* defines attributes via import arguments
* generates read-write accessors
* supports lazy attribute defaults
* supports custom accessors
* superclass provides a standard new constructor
* new takes a hash reference or list of key/value pairs
* new has heuristics to catch constructor attribute typos
* new calls BUILD for each class from parent to child
* superclass provides a DESTROY method
* DESTROY calls DEMOLISH for each class from child to parent

Fedora Account System Username: jplesnik

Comment 1 Paul Howarth 2013-09-16 14:08:13 UTC
Since you're not including the dependencies for old Perls, you should probably BR: perl >= 5.14 rather than 5.8.1

Comment 2 Petr Šabata 2013-09-16 15:39:30 UTC
The package is fine except for the bit pointed out by Paul.
I'd suggest wrapping the dependency (both BR and the filter) in a condition checking for the version of perl available.

Comment 3 Paul Howarth 2013-09-16 15:54:10 UTC
My local build has this:

%if %(perl -e 'print $] < 5.014 ? 1 : 0;')
BuildRequires:  perl(Devel::GlobalDestruction)
%else
%global __requires_exclude ^perl\\(Devel::GlobalDestruction\\)
%endif

I have a similar condition for the Perl < 5.10 support but Class-Tiny seems to have issues there (https://github.com/dagolden/Class-Tiny/issues/8) and a test build of Pod-Spell 1.07 on 5.8.8 failed for me with an error that suggested it was a Class::Tiny problem, whilst 5.10 passed OK. So I'd be inclined not to even try to support Perl < 5.10 at the moment.

Comment 5 Petr Šabata 2013-09-16 16:43:05 UTC
Well, additional minimum perl depdency or Class::Tiny fixes could be added later if we hit the problems you're referring to.

I don't think that's a review blocker.
Approving.

Comment 6 Jitka Plesnikova 2013-09-17 06:14:06 UTC
New Package SCM Request
=======================
Package Name: perl-Class-Tiny
Short Description: Minimalist class construction
Owners: jplesnik ppisar psabata
Branches: f20
InitialCC: perl-sig

Comment 7 Gwyn Ciesla 2013-09-17 12:44:56 UTC
Git done (by process-git-requests).

Comment 8 Fedora Update System 2013-09-17 13:37:53 UTC
perl-Class-Tiny-0.008-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/perl-Class-Tiny-0.008-1.fc20

Comment 9 Paul Howarth 2013-09-17 13:46:40 UTC
Class-Tiny 0.009 is out now by the way.

Comment 10 Fedora Update System 2013-09-17 18:14:19 UTC
Package perl-Class-Tiny-0.008-1.fc20:
* should fix your issue,
* was pushed to the Fedora 20 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing perl-Class-Tiny-0.008-1.fc20'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2013-16942/perl-Class-Tiny-0.008-1.fc20
then log in and leave karma (feedback).

Comment 11 Fedora Update System 2013-09-18 06:37:46 UTC
perl-Class-Tiny-0.009-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/perl-Class-Tiny-0.009-1.fc20

Comment 12 Fedora Update System 2013-09-27 00:49:10 UTC
perl-Class-Tiny-0.009-1.fc20 has been pushed to the Fedora 20 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.