https://bugzilla.redhat.com/show_bug.cgi?id=1466271
Petr Pisar <ppi...@redhat.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+ --- Comment #1 from Petr Pisar <ppi...@redhat.com> --- URL and Source0 addresses are Ok. Source0 archive (SHA-256: 6381322ec44cf28e2ef693a86fed4cc2d706edd28d073c5d90cb0a5137459cb6) is original. Ok. Summary verified from lib/Mail/Box/POP3.pod. Ok. Description verified from lib/Mail/Box/POP3.pod. Ok. License verified in lib/Mail/Box/POP3.pod, lib/Mail/Box/POP3/Message.pod, lib/Mail/Box/POP3s.pod, and lib/Mail/Transport/POP3.pod. Ok. No XS code, noarch BuildArch is Ok. FIX: Do not build-require `perl(Mail::Reporter)'. It's not used. FIX: Build-require `perl(IO::Socket::INET)' (lib/Mail/Transport/POP3.pm:260). TODO: IO::Socket::INET does not support IPv6. Please patch the code to use IO::Socket::IP instead. All test pass. Ok. $ rpmlint perl-Mail-Box-POP3.spec ../SRPMS/perl-Mail-Box-POP3-3.001-1.fc27.src.rpm ../RPMS/noarch/perl-Mail-Box-POP3-3.001-1.fc27.noarch.rpm perl-Mail-Box-POP3.src: W: spelling-error %description -l en_US focusses -> focuses, focused, cusses perl-Mail-Box-POP3.noarch: W: spelling-error %description -l en_US focusses -> focuses, focused, cusses 2 packages and 1 specfiles checked; 0 errors, 2 warnings. FIX: Correct the misspelling. $ rpm -q -lv -p ../RPMS/noarch/perl-Mail-Box-POP3-3.001-1.fc27.noarch.rpm drwxr-xr-x 2 root root 0 Jun 30 09:45 /usr/share/doc/perl-Mail-Box-POP3 -rw-r--r-- 1 root root 281 Feb 2 16:07 /usr/share/doc/perl-Mail-Box-POP3/ChangeLog -rw-r--r-- 1 root root 811 Feb 2 16:07 /usr/share/doc/perl-Mail-Box-POP3/README -rw-r--r-- 1 root root 8313 Jun 30 09:45 /usr/share/man/man3/Mail::Box::POP3.3pm.gz -rw-r--r-- 1 root root 6055 Jun 30 09:45 /usr/share/man/man3/Mail::Box::POP3::Message.3pm.gz -rw-r--r-- 1 root root 6641 Jun 30 09:45 /usr/share/man/man3/Mail::Box::POP3s.3pm.gz -rw-r--r-- 1 root root 6276 Jun 30 09:45 /usr/share/man/man3/Mail::Transport::POP3.3pm.gz drwxr-xr-x 2 root root 0 Jun 30 09:45 /usr/share/perl5/vendor_perl/Mail drwxr-xr-x 2 root root 0 Jun 30 09:45 /usr/share/perl5/vendor_perl/Mail/Box drwxr-xr-x 2 root root 0 Jun 30 09:45 /usr/share/perl5/vendor_perl/Mail/Box/POP3 -rw-r--r-- 1 root root 5116 Feb 2 16:07 /usr/share/perl5/vendor_perl/Mail/Box/POP3.pm -rw-r--r-- 1 root root 19832 Feb 2 16:07 /usr/share/perl5/vendor_perl/Mail/Box/POP3.pod -rw-r--r-- 1 root root 1808 Feb 2 16:07 /usr/share/perl5/vendor_perl/Mail/Box/POP3/Message.pm -rw-r--r-- 1 root root 16411 Feb 2 16:07 /usr/share/perl5/vendor_perl/Mail/Box/POP3/Message.pod -rw-r--r-- 1 root root 1572 Feb 2 16:07 /usr/share/perl5/vendor_perl/Mail/Box/POP3/Test.pm -rw-r--r-- 1 root root 648 Feb 2 16:07 /usr/share/perl5/vendor_perl/Mail/Box/POP3s.pm -rw-r--r-- 1 root root 15698 Feb 2 16:07 /usr/share/perl5/vendor_perl/Mail/Box/POP3s.pod drwxr-xr-x 2 root root 0 Jun 30 09:45 /usr/share/perl5/vendor_perl/Mail/Transport -rw-r--r-- 1 root root 9381 Feb 2 16:07 /usr/share/perl5/vendor_perl/Mail/Transport/POP3.pm -rw-r--r-- 1 root root 13090 Feb 2 16:07 /usr/share/perl5/vendor_perl/Mail/Transport/POP3.pod File layout and permissions are Ok. $ rpm -q --requires -p ../RPMS/noarch/perl-Mail-Box-POP3-3.001-1.fc27.noarch.rpm | sort -f | uniq -c 1 perl(:MODULE_COMPAT_5.26.0) 1 perl(base) 1 perl(Carp) 1 perl(Digest::MD5) 1 perl(Exporter) 1 perl(File::Basename) 1 perl(File::Spec) 1 perl(IO::Socket) 1 perl(List::Util) 1 perl(Mail::Box::FastScalar) >= 3 1 perl(Mail::Box::Net) >= 3 1 perl(Mail::Box::Net::Message) 1 perl(Mail::Box::Parser::Perl) >= 3 1 perl(Mail::Box::POP3) 1 perl(Mail::Box::POP3::Message) 1 perl(Mail::Transport::POP3) 1 perl(Mail::Transport::Receive) >= 3 1 perl(Socket) 1 perl(strict) 1 perl(vars) 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 FIX: Run-require `perl(IO::Socket::SSL)' and `perl(IO::Socket::INET)' (lib/Mail/Transport/POP3.pm:260). $ rpm -q --provides -p ../RPMS/noarch/perl-Mail-Box-POP3-3.001-1.fc27.noarch.rpm | sort -f | uniq -c 1 perl(Mail::Box::POP3) = 3.001 1 perl(Mail::Box::POP3::Message) = 3.001 1 perl(Mail::Box::POP3::Test) = 3.001 1 perl(Mail::Box::POP3s) = 3.001 1 perl(Mail::Transport::POP3) = 3.001 1 perl-Mail-Box-POP3 = 3.001-1.fc27 Binary provides are Ok. $ resolvedeps rawhide ../RPMS/noarch/perl-Mail-Box-POP3-3.001-1.fc27.noarch.rpm Binary dependencies resolvable. Ok. Package builds in F27 (https://koji.fedoraproject.org/koji/taskinfo?taskID=20259573). Ok. Otherwise the package is in line with Fedora and Perl packaging guidelines. Please correct all `FIX' items and consider fixing the `TODO' item 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@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org