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

Reply via email to