Bug 1139009 - Review Request: perl-LV - Perl module to make lvalue subroutines easy and practical
Summary: Review Request: perl-LV - Perl module to make lvalue subroutines easy and pra...
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: 1139007
Blocks: 858586
TreeView+ depends on / blocked
 
Reported: 2014-09-07 12:35 UTC by Emmanuel Seyman
Modified: 2014-09-14 01:08 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-09-14 01:08:06 UTC
Type: ---
Embargoed:
psabata: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Emmanuel Seyman 2014-09-07 12:35:43 UTC
Spec URL: http://people.parinux.org/~seyman/fedora/perl-LV/perl-LV.spec
SRPM URL: http://people.parinux.org/~seyman/fedora/perl-LV/perl-LV-0.006-1.fc20.src.rpm

Description:
This module makes lvalue subroutines easy and practical to use. It's
inspired by the lvalue module which is sadly problematic because of the
existence of another module on CPAN called Lvalue.

Fedora Account System Username: eseyman

Rpmlint output : 1 packages and 1 specfiles checked; 0 errors, 2 warnings.

Comment 1 Petr Šabata 2014-09-08 14:37:38 UTC
Could we get a more descriptive summary?

You could substitute PERL_INSTALL_ROOT with the standard DESTDIR.

You could drop line 41; it's unnecessary nowadays.

You're missing some BRs, namely:
 - Carp, lib/LV/Backend/Magic.pm:10, via 04magic.t
 - Exporter, lib/LV.pm:16, via 01basic.t
 - strict, used by everything
 - Variable::Magic, lib/LV/Backend/Magic.pm:11, via 04magic.t -> lib/LV.pm:46
 - warnings, used by everything

Both 03sentinel.t and 04magic.t will skip their plans if the backend modules cannot be loaded, therefore Carp, Sentinel and Variable::Magic are optional.  However, it'd be a good idea to run those tests anyway.

Comment 2 Emmanuel Seyman 2014-09-10 09:12:06 UTC
(In reply to Petr Šabata from comment #1)
>
> Could we get a more descriptive summary?

Done.

> You could substitute PERL_INSTALL_ROOT with the standard DESTDIR.

Done.

> You could drop line 41; it's unnecessary nowadays.

Line 36 removed.

> You're missing some BRs, namely:
>  - Carp, lib/LV/Backend/Magic.pm:10, via 04magic.t
>  - Exporter, lib/LV.pm:16, via 01basic.t
>  - strict, used by everything
>  - Variable::Magic, lib/LV/Backend/Magic.pm:11, via 04magic.t -> lib/LV.pm:46
>  - warnings, used by everything

All added.

Spec URL: http://people.parinux.org/~seyman/fedora/perl-LV/perl-LV.spec
SRPM URL: http://people.parinux.org/~seyman/fedora/perl-LV/perl-LV-0.006-2.fc20.src.rpm

Comment 3 Petr Šabata 2014-09-10 11:51:35 UTC
Okay, approving.

Comment 4 Emmanuel Seyman 2014-09-11 05:37:10 UTC
New Package SCM Request
=======================
Package Name: perl-LV
Short Description: Perl module to make lvalue subroutines easy and practical
Upstream URL: http://search.cpan.org/dist/LV/
Owners: eseyman
Branches: f21 f20
InitialCC: perl-sig

Comment 5 Gwyn Ciesla 2014-09-11 12:02:10 UTC
Git done (by process-git-requests).

Comment 6 Emmanuel Seyman 2014-09-14 01:08:06 UTC
Built in Rawhide.


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