https://bugzilla.redhat.com/show_bug.cgi?id=1421582 Petr Pisar <ppisar@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+ --- Comment #1 from Petr Pisar <ppisar@xxxxxxxxxx> --- URL and Source0 addresses are usable. Ok. Source archive (SHA-256: de99ce915ab0f93781580e3433d7dc9c2585a2789627173b280c6030ac0b1953) is original. Ok. Summary verified from lib/namespace/sweep.pm. Ok. Description verified from lib/namespace/sweep.pm. Ok. License verified from lib/namespace/sweep.pm, LICENSE, README. Ok. No XS code, noarch BuildArch is Ok. TODO: Constrain `perl(ExtUtils::MakeMaker)' dependency with `>= 6.30' (Makefile.PL:7). FIX: Build-require `perl(overload)' (t/overload.t:10). All tests pass. Ok. $ rpmlint perl-namespace-sweep.spec ../SRPMS/perl-namespace-sweep-0.006-1.fc26.src.rpm ../RPMS/noarch/perl-namespace-sweep-0.006-1.fc26.noarch.rpm perl-namespace-sweep.spec:29: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 29) perl-namespace-sweep.src: W: spelling-error %description -l en_US pragma -> pragmatic perl-namespace-sweep.src:29: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 29) perl-namespace-sweep.noarch: W: spelling-error %description -l en_US pragma -> pragmatic 2 packages and 1 specfiles checked; 0 errors, 4 warnings. TODO: Normalize spaces. $ rpm -q -lv -p ../RPMS/noarch/perl-namespace-sweep-0.006-1.fc26.noarch.rpm drwxr-xr-x 2 root root 0 Feb 23 15:31 /usr/share/doc/perl-namespace-sweep -rw-r--r-- 1 root root 308 Oct 16 2012 /usr/share/doc/perl-namespace-sweep/README drwxr-xr-x 2 root root 0 Feb 23 15:31 /usr/share/licenses/perl-namespace-sweep -rw-r--r-- 1 root root 18350 Oct 16 2012 /usr/share/licenses/perl-namespace-sweep/LICENSE -rw-r--r-- 1 root root 2733 Feb 23 15:31 /usr/share/man/man3/namespace::sweep.3pm.gz drwxr-xr-x 2 root root 0 Feb 23 15:31 /usr/share/perl5/vendor_perl/namespace -rw-r--r-- 1 root root 6670 Oct 16 2012 /usr/share/perl5/vendor_perl/namespace/sweep.pm File layout and permissions are Ok. $ rpm -q --requires -p ../RPMS/noarch/perl-namespace-sweep-0.006-1.fc26.noarch.rpm | sort -f | uniq -c 1 perl(:MODULE_COMPAT_5.24.1) 1 perl(B::Hooks::EndOfScope) >= 0.09 1 perl(Carp) 1 perl(Data::Dumper) 1 perl(List::Util) 1 perl(Package::Stash) >= 0.33 1 perl(Scalar::Util) 1 perl(strict) 1 perl(Sub::Identify) >= 0.04 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(PayloadIsXz) <= 5.2-1 Binary requires are Ok. $ rpm -q --provides -p ../RPMS/noarch/perl-namespace-sweep-0.006-1.fc26.noarch.rpm | sort -f | uniq -c 1 perl(namespace::sweep) = 0.006 1 perl-namespace-sweep = 0.006-1.fc26 Binary provides are Ok. $ resolvedeps rawhide ../RPMS/noarch/perl-namespace-sweep-0.006-1.fc26.noarch.rpm Binary dependencies resolvable. Ok. Package builds in F26 (https://koji.fedoraproject.org/koji/taskinfo?taskID=18013656). Ok. Otherwise the package is in line with Fedora and Perl packaging guidelines. Please correct the `FIX' item and consider fixing all the `TODO' items before building this package. Resolution: Package APPROVED. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx