https://bugzilla.redhat.com/show_bug.cgi?id=1465817
Petr Pisar <ppisar(a)redhat.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flags|fedora-review? |fedora-review+
--- Comment #1 from Petr Pisar <ppisar(a)redhat.com> ---
URL and Source0 addresses are Ok.
Source archive (SHA-256:
48315c3716c66d86ef6dbd766a77d3f19f7d95bc47799bd0f0bc33ffe3563f05) is original.
Ok.
TODO: The package delivers more than Mail::Box::IMAP4. E.g. there is
Mail::Server::IMAP4. Thus the summary and the description does not match the
whole content. Either make the text more general or split the modules into
subpackages with appropriate texts. Also the last "See also
Mail::Server::IMAP4" sentence is not helpful because Mail::Server::IMAP4 is
provided by this package.
TODO: I recommend to subpackage Mail::Server::IMAP4* modules.
License verified from lib/Mail/Transport/IMAP4.pod, lib/Mail/Box/IMAP4.pod,
lib/Mail/Box/IMAP4/Message.pod, lib/Mail/Box/IMAP4/Head.pod,
lib/Mail/Server/IMAP4.pod, lib/Mail/Server/IMAP4/Fetch.pod,
lib/Mail/Server/IMAP4/List.pod, lib/Mail/Server/IMAP4/User.pod, and
lib/Mail/Server/IMAP4/Search.pod. Ok.
No XS code, noarch BuildArch is Ok.
TODO: Build-require `perl(:VERSION) >= 5.10' (Makefile.PL:3).
FIX: Do not build-require `perl(Mail::Server)'. The only occurrence is at
lib/Mail/Server/IMAP4.pm:13 and this is not used when running the tests.
All tests pass. Ok.
$ rpmlint perl-Mail-Box-IMAP4.spec
../SRPMS/perl-Mail-Box-IMAP4-3.001-1.fc27.src.rpm
../RPMS/noarch/perl-Mail-Box-IMAP4-3.001-1.fc27.noarch.rpm
2 packages and 1 specfiles checked; 0 errors, 0 warnings.
rpmlint is Ok.
$ rpm -q -lv -p ../RPMS/noarch/perl-Mail-Box-IMAP4-3.001-1.fc27.noarch.rpm
drwxr-xr-x 2 root root 0 Jun 28 15:03
/usr/share/doc/perl-Mail-Box-IMAP4
-rw-r--r-- 1 root root 254 Feb 2 16:02
/usr/share/doc/perl-Mail-Box-IMAP4/ChangeLog
-rw-r--r-- 1 root root 817 Feb 2 16:02
/usr/share/doc/perl-Mail-Box-IMAP4/README
-rw-r--r-- 1 root root 9199 Jun 28 15:03
/usr/share/man/man3/Mail::Box::IMAP4.3pm.gz
-rw-r--r-- 1 root root 3715 Jun 28 15:03
/usr/share/man/man3/Mail::Box::IMAP4::Head.3pm.gz
-rw-r--r-- 1 root root 8089 Jun 28 15:03
/usr/share/man/man3/Mail::Box::IMAP4::Message.3pm.gz
-rw-r--r-- 1 root root 2388 Jun 28 15:03
/usr/share/man/man3/Mail::Server::IMAP4.3pm.gz
-rw-r--r-- 1 root root 3040 Jun 28 15:03
/usr/share/man/man3/Mail::Server::IMAP4::Fetch.3pm.gz
-rw-r--r-- 1 root root 3252 Jun 28 15:03
/usr/share/man/man3/Mail::Server::IMAP4::List.3pm.gz
-rw-r--r-- 1 root root 3314 Jun 28 15:03
/usr/share/man/man3/Mail::Server::IMAP4::Search.3pm.gz
-rw-r--r-- 1 root root 6263 Jun 28 15:03
/usr/share/man/man3/Mail::Server::IMAP4::User.3pm.gz
-rw-r--r-- 1 root root 7080 Jun 28 15:03
/usr/share/man/man3/Mail::Transport::IMAP4.3pm.gz
drwxr-xr-x 2 root root 0 Jun 28 15:03
/usr/share/perl5/vendor_perl/Mail
drwxr-xr-x 2 root root 0 Jun 28 15:03
/usr/share/perl5/vendor_perl/Mail/Box
drwxr-xr-x 2 root root 0 Jun 28 15:03
/usr/share/perl5/vendor_perl/Mail/Box/IMAP4
-rw-r--r-- 1 root root 8198 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Box/IMAP4.pm
-rw-r--r-- 1 root root 22766 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Box/IMAP4.pod
-rw-r--r-- 1 root root 1308 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Box/IMAP4/Head.pm
-rw-r--r-- 1 root root 7541 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Box/IMAP4/Head.pod
-rw-r--r-- 1 root root 3917 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Box/IMAP4/Message.pm
-rw-r--r-- 1 root root 21656 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Box/IMAP4/Message.pod
drwxr-xr-x 2 root root 0 Jun 28 15:03
/usr/share/perl5/vendor_perl/Mail/Server
drwxr-xr-x 2 root root 0 Jun 28 15:03
/usr/share/perl5/vendor_perl/Mail/Server/IMAP4
-rw-r--r-- 1 root root 487 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Server/IMAP4.pm
-rw-r--r-- 1 root root 2757 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Server/IMAP4.pod
-rw-r--r-- 1 root root 7744 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Server/IMAP4/Fetch.pm
-rw-r--r-- 1 root root 3696 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Server/IMAP4/Fetch.pod
-rw-r--r-- 1 root root 2920 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Server/IMAP4/List.pm
-rw-r--r-- 1 root root 4661 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Server/IMAP4/List.pod
-rw-r--r-- 1 root root 444 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Server/IMAP4/Search.pm
-rw-r--r-- 1 root root 5631 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Server/IMAP4/Search.pod
-rw-r--r-- 1 root root 3199 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Server/IMAP4/User.pm
-rw-r--r-- 1 root root 14452 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Server/IMAP4/User.pod
drwxr-xr-x 2 root root 0 Jun 28 15:03
/usr/share/perl5/vendor_perl/Mail/Transport
-rw-r--r-- 1 root root 13393 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Transport/IMAP4.pm
-rw-r--r-- 1 root root 14399 Feb 2 16:02
/usr/share/perl5/vendor_perl/Mail/Transport/IMAP4.pod
File layout and permissions are Ok.
$ rpm -q --req[test@fedora-27 SPECS]$ rpm -q --requires -p
../RPMS/noarch/perl-Mail-Box-IMAP4-3.001-1.fc27.noarch.rpm | sort -f | uniq -c
1 perl(:MODULE_COMPAT_5.26.0)
1 perl(base)
1 perl(Date::Parse)
1 perl(Digest::HMAC_MD5)
1 perl(Digest::MD5)
1 perl(List::Util)
1 perl(Mail::Box::IMAP4::Head)
1 perl(Mail::Box::IMAP4::Message)
1 perl(Mail::Box::Manage::User) >= 3
1 perl(Mail::Box::Net) >= 3
1 perl(Mail::Box::Net::Message)
1 perl(Mail::Box::Parser::Perl)
1 perl(Mail::Box::Search) >= 3
1 perl(Mail::IMAPClient)
1 perl(Mail::Message) >= 3
1 perl(Mail::Message::Body::Lines) >= 3
1 perl(Mail::Message::Head)
1 perl(Mail::Message::Head::Complete) >= 3
1 perl(Mail::Message::Head::Delayed) >= 3
1 perl(Mail::Server) >= 3
1 perl(Mail::Server::IMAP4::Fetch)
1 perl(Mail::Server::IMAP4::List)
1 perl(Mail::Server::IMAP4::Search)
1 perl(Mail::Transport::IMAP4)
1 perl(Mail::Transport::Receive) >= 3
1 perl(Scalar::Util)
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: The `Mail::Message' is not used directly anywhere in the library. Do not
run-require it.
Mail::Message::Body::Lines is used from lib/Mail/Box/IMAP4.pm:56. Ok.
$ rpm -q --provides -p
../RPMS/noarch/perl-Mail-Box-IMAP4-3.001-1.fc27.noarch.rpm | sort -f | uniq -c
1 perl(Mail::Box::IMAP4) = 3.001
1 perl(Mail::Box::IMAP4::Head) = 3.001
1 perl(Mail::Box::IMAP4::Message) = 3.001
1 perl(Mail::Server::IMAP4) = 3.001
1 perl(Mail::Server::IMAP4::Fetch) = 3.001
1 perl(Mail::Server::IMAP4::List) = 3.001
1 perl(Mail::Server::IMAP4::Search) = 3.001
1 perl(Mail::Server::IMAP4::User) = 3.001
1 perl(Mail::Transport::IMAP4) = 3.001
1 perl-Mail-Box-IMAP4 = 3.001-1.fc27
Binary provides are Ok.
$ resolvedeps rawhide
../RPMS/noarch/perl-Mail-Box-IMAP4-3.001-1.fc27.noarch.rpm
Binary dependencies resolvable. Ok.
Package builds in F27
(
https://koji.fedoraproject.org/koji/taskinfo?taskID=20226839).
Otherwise the package is in line with Fedora and Perl packaging guidelines.
Please correct the `FIX' items and consider fixing `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