Bug 1858449 - Review Request: perl-MooX-TypeTiny - Optimized type checks for Moo + Type::Tiny
Summary: Review Request: perl-MooX-TypeTiny - Optimized type checks for Moo + Type::Tiny
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Petr Pisar
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: FE-DEADREVIEW 1858466
TreeView+ depends on / blocked
 
Reported: 2020-07-18 04:30 UTC by Ralf Corsepius
Modified: 2021-08-21 00:45 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-08-21 00:45:31 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Ralf Corsepius 2020-07-18 04:30:11 UTC
Spec URL: https://corsepiu.fedorapeople.org/packages/perl-MooX-TypeTiny.spec
SRPM URL: https://corsepiu.fedorapeople.org/packages/perl-MooX-TypeTiny-0.002003-1.fc33.src.rpm
Description:
This module optimizes Moo type checks when used with Type::Tiny to perform
better. It will automatically apply to isa checks and coercions that use
Type::Tiny. Non-Type::Tiny isa checks will work as normal.

Fedora Account System Username: corsepiu

Comment 1 Petr Pisar 2020-07-20 13:52:34 UTC
URL and Source addresses are Ok.
TODO: Remove a trailing slash from the URL value.

Source0 archive (SHA-256: d81e26ff6f8db10261f0087f96dc54367dcb49a9f3de8d53238f834ece19624b) is original. Ok.
Summary verified from lib/MooX/TypeTiny.pm. Ok.
Description verified from lib/MooX/TypeTiny.pm. Ok.
License verified from Makefile.PL, lib/MooX/TypeTiny.pm, LICENSE, and README. Ok.
No XS code, noarch BuildArch is Ok.

FIX: Build-require "perl-interpreter" (perl-MooX-TypeTiny.spec:30).
FIX: Build-require "make" (perl-MooX-TypeTiny.spec:31).
FIX: Build-require "coreutils" (perl-MooX-TypeTiny.spec:39).

TODO: Use %{make_build} and %{make_install} instead of "make %{?_smp_mflags}" and "make pure_install".
TODO: Remove "find $RPM_BUILD_ROOT -depth -type d -exec rmdir {} 2>/dev/null \;". It's not needed.

FIX: Build-require "perl(strict)" (Makefile.PL:1).
FIX: Build-require "perl(warnings)" (Makefile.PL:2).
TODO: Build-require "perl(:VERSION) >= 5.6" (Makefile.PL:3).

Moo::_Utils is a private module. Ok.

FIX: Build-require "perl(Moo::Role)" (lib/MooX/TypeTiny/Role/GenerateAccessor.pm:2).
FIX: Build-require "perl(Scalar::Util)" (lib/MooX/TypeTiny/Role/GenerateAccessor.pm:4).
FIX: Build-require "perl(Sub::Quote)" (lib/MooX/TypeTiny/Role/GenerateAccessor.pm:3).
FIX: Build-require "perl(Types::Standard)" (t/basic.t:7).

All tests pass. Ok.

$ rpmlint perl-MooX-TypeTiny.spec ../SRPMS/perl-MooX-TypeTiny-0.002003-1.fc33.src.rpm ../RPMS/noarch/perl-MooX-TypeTiny-0.002003-1.fc33.noarch.rpm 
sh: /usr/bin/python2: No such file or directory
perl-MooX-TypeTiny.src: W: spelling-error %description -l en_US isa -> is, visa, Lisa
perl-MooX-TypeTiny.src: W: spelling-error %description -l en_US coercions -> coercion, coercion's, coercion s
perl-MooX-TypeTiny.noarch: W: spelling-error %description -l en_US isa -> is, visa, Lisa
perl-MooX-TypeTiny.noarch: W: spelling-error %description -l en_US coercions -> coercion, coercion's, coercion s
2 packages and 1 specfiles checked; 0 errors, 4 warnings.
rpmlint is Ok.

$ rpm -q -lv -p ../RPMS/noarch/perl-MooX-TypeTiny-0.002003-1.fc33.noarch.rpm 
drwxr-xr-x    2 root     root                        0 Jul 20 15:36 /usr/share/doc/perl-MooX-TypeTiny
-rw-r--r--    1 root     root                      970 Apr 29 16:35 /usr/share/doc/perl-MooX-TypeTiny/Changes
-rw-r--r--    1 root     root                     1186 Apr 29 16:35 /usr/share/doc/perl-MooX-TypeTiny/README
drwxr-xr-x    2 root     root                        0 Jul 20 15:36 /usr/share/licenses/perl-MooX-TypeTiny
-rw-r--r--    1 root     root                    18235 Apr 29 16:35 /usr/share/licenses/perl-MooX-TypeTiny/LICENSE
-rw-r--r--    1 root     root                     1711 Jul 20 15:36 /usr/share/man/man3/MooX::TypeTiny.3pm.gz
drwxr-xr-x    2 root     root                        0 Jul 20 15:36 /usr/share/perl5/vendor_perl/MooX
drwxr-xr-x    2 root     root                        0 Jul 20 15:36 /usr/share/perl5/vendor_perl/MooX/TypeTiny
-rw-r--r--    1 root     root                     1856 Apr 29 16:35 /usr/share/perl5/vendor_perl/MooX/TypeTiny.pm
drwxr-xr-x    2 root     root                        0 Jul 20 15:36 /usr/share/perl5/vendor_perl/MooX/TypeTiny/Role
-rw-r--r--    1 root     root                     2029 Apr 26 06:40 /usr/share/perl5/vendor_perl/MooX/TypeTiny/Role/GenerateAccessor.pm
File layout and permissions are Ok.

$ rpm -q --requires -p ../RPMS/noarch/perl-MooX-TypeTiny-0.002003-1.fc33.noarch.rpm | sort -f | uniq -c
      1 perl(Moo::Role)
      1 perl(Scalar::Util)
      1 perl(strict)
      1 perl(Sub::Quote)
      1 perl(warnings)
      1 rpmlib(CompressedFileNames) <= 3.0.4-1
      1 rpmlib(FileDigests) <= 4.6.0-1
      1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
      1 rpmlib(PayloadIsZstd) <= 5.4.18-1
FIX: Run-require "perl(:MODULE_COMPAT...)".
FIX: Run-require "perl(Moo) >= 2.004)" (lib/MooX/TypeTiny.pm:11).

$ rpm -q --provides -p ../RPMS/noarch/perl-MooX-TypeTiny-0.002003-1.fc33.noarch.rpm | sort -f | uniq -c
      1 perl(MooX::TypeTiny) = 0.002003
      1 perl(MooX::TypeTiny::Role::GenerateAccessor)
      1 perl-MooX-TypeTiny = 0.002003-1.fc33
Binary provides are Ok.

$ resolvedeps rawhide ../RPMS/noarch/perl-MooX-TypeTiny-0.002003-1.fc33.noarch.rpm 
Binary dependencies are resolvable. Ok.

The package builds in Fedora 33 (https://koji.fedoraproject.org/koji/taskinfo?taskID=47496724). Ok.

Otherwise the package is in line with Fedora and Perl pacackaging guidelines.
Please correct the "FIX" items, consider fixing "TODO" items, and provide a new spec file.
Resolution: Package NOT approved.

Comment 2 Package Review 2021-07-21 00:45:20 UTC
This is an automatic check from review-stats script.

This review request ticket hasn't been updated for some time, but it seems
that the review is still being working out by you. If this is right, please
respond to this comment clearing the NEEDINFO flag and try to reach out the
submitter to proceed with the review.

If you're not interested in reviewing this ticket anymore, please clear the
fedora-review flag and reset the assignee, so that a new reviewer can take
this ticket.

Without any reply, this request will shortly be resetted.

Comment 3 Petr Pisar 2021-07-21 08:57:51 UTC
Ralf, any progress with this package?

Comment 4 Package Review 2021-08-21 00:45:31 UTC
This is an automatic action taken by review-stats script.

The ticket submitter failed to clear the NEEDINFO flag in a month.
As per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews
we consider this ticket as DEADREVIEW and proceed to close it.


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