Bug 1465817 - Review Request: perl-Mail-Box-IMAP4 - Handle IMAP4 folders as client
Summary: Review Request: perl-Mail-Box-IMAP4 - Handle IMAP4 folders as client
Keywords:
Status: CLOSED ERRATA
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:
TreeView+ depends on / blocked
 
Reported: 2017-06-28 09:38 UTC by Jitka Plesnikova
Modified: 2017-07-07 23:05 UTC (History)
2 users (show)

Fixed In Version: perl-Mail-Box-IMAP4-3.001-1.fc27
Clone Of:
Environment:
Last Closed: 2017-07-07 23:05:48 UTC
Type: ---
Embargoed:
ppisar: fedora-review+


Attachments (Terms of Use)

Description Jitka Plesnikova 2017-06-28 09:38:54 UTC
Spec URL: https://jplesnik.fedorapeople.org/perl-Mail-Box-IMAP4/perl-Mail-Box-IMAP4.spec
SRPM URL: https://jplesnik.fedorapeople.org/perl-Mail-Box-IMAP4/perl-Mail-Box-IMAP4-3.001-1.fc27.src.rpm
Description:
Maintain a folder which has its messages stored on a remote server. The
communication between the client application and the server is implemented
using the IMAP4 protocol. See also Mail::Server::IMAP4.

Fedora Account System Username: jplesnik

The package was split from perl-Mail-Box.

Comment 1 Petr Pisar 2017-06-28 13:32:55 UTC
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.

Comment 2 Gwyn Ciesla 2017-06-28 14:46:24 UTC
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/perl-Mail-Box-IMAP4

Comment 3 Jitka Plesnikova 2017-06-29 10:53:22 UTC
Thank you for the review and the repository.

Comment 4 Fedora Update System 2017-06-29 11:13:29 UTC
perl-Mail-Box-IMAP4-3.001-1.fc26 has been submitted as an update to Fedora 26. https://bodhi.fedoraproject.org/updates/FEDORA-2017-888a373415

Comment 5 Fedora Update System 2017-06-29 23:25:54 UTC
perl-Mail-Box-IMAP4-3.001-1.fc26 has been pushed to the Fedora 26 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-888a373415

Comment 6 Fedora Update System 2017-06-30 06:58:17 UTC
perl-Mail-Box-IMAP4-3.002-1.fc26 has been submitted as an update to Fedora 26. https://bodhi.fedoraproject.org/updates/FEDORA-2017-d90a2aeeba

Comment 7 Fedora Update System 2017-06-30 20:26:09 UTC
perl-Mail-Box-IMAP4-3.002-1.fc26 has been pushed to the Fedora 26 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-d90a2aeeba

Comment 8 Fedora Update System 2017-07-07 23:05:48 UTC
perl-Mail-Box-IMAP4-3.002-1.fc26 has been pushed to the Fedora 26 stable repository. If problems still persist, please make note of it in this bug report.


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