Bug 556404 - Merge Review: perl-MIME-Lite - low-calorie MIME generator
Summary: Merge Review: perl-MIME-Lite - low-calorie MIME generator
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Jaroslav Škarvada
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-01-18 09:39 UTC by Marcela Mašláňová
Modified: 2010-02-04 08:53 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2010-02-04 08:53:43 UTC
Type: ---
Embargoed:
jskarvad: fedora-review+


Attachments (Terms of Use)

Description Marcela Mašláňová 2010-01-18 09:39:02 UTC
SRPM: http://kojipkgs.fedoraproject.org/packages/perl-MIME-Lite/3.027/2.fc13/src/perl-MIME-Lite-3.027-2.fc13.src.rpm
SPEC: http://koji.fedoraproject.org/koji/fileinfo?rpmID=1712343&filename=perl-MIME-Lite.spec
Description: MIME::Lite is intended as a simple, standalone module for generating (not parsing!) MIME messages... specifically, it allows you to output a simple, decent single- or multi-part message with text or binaryattachments. It does not require that you have the Mail:: or MIME:: modules installed.

Comment 1 Jaroslav Škarvada 2010-01-27 09:10:20 UTC
MUST items:
[NO] rpmplint is silent
perl-MIME-Lite.spec: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 14)
0 packages and 1 specfiles checked; 0 errors, 1 warnings.

[YES] Package meets naming guidelines.
[YES] Package meets packaging guidelines.
[YES] Spec file matches base package name.
[NO] License file is present, matching with spec file.
Comment 1)

[YES] Licensing Guidelines are met.
[YES] Spec file is legible and in American English.
Comment 2)

[YES] Sources match upstream.
[YES] Package builds OK.
[YES] BuildRequires is correct.
[YES] Package doesn't bundle copies of system libraries.
[YES] Package owns all the directories it creates.
[NO] Package has no duplicity in %files.
Comment 3)

[YES] Permission on files are set properly.
[YES] %clean section is correct.
[YES] Spec file has consistant macro usage.
[YES] Package is code or permissable content.
[YES] %doc files don't affect runtime.
[YES] No .la libtool archives.
[YES][Perl] Package doesn't own files/directories that other packages own.
[YES] Package has rm -rf $RPM_BUILD_ROOT at beginning of %install.
[YES] Spec file is valid UTF-8.

Should items:
[YES] Package builds in mock.
[YES] Package uses sane scriptlets.
[YES] Package contains man pages.


**Comments:
1) COPYING and LICENSE files are not packaged into RPM.

2) In %description second sentence should start with capital letter. There is extra space after second sentence. 

3) changes.pod is installed two times:
-rw-r--r--  /usr/share/doc/perl-MIME-Lite-3.027/changes.pod
-r--r--r--  /usr/share/perl5/MIME/changes.pod

Comment 2 Marcela Mašláňová 2010-01-27 10:24:15 UTC
1/ added
2/ fixed
3/ fixed

But rpmlint got fixed! More errors was produced:
[marca@caladan devel]$ rpmlint noarch/perl-MIME-Lite-3.027-3.fc13.noarch.rpm
perl-MIME-Lite.noarch: W: spelling-error %description -l en_US multi -> mulch, mufti
- multi is okay.
perl-MIME-Lite.noarch: W: spurious-executable-perm /usr/share/doc/perl-MIME-Lite-3.027/examples/mime_fwd
perl-MIME-Lite.noarch: W: spurious-executable-perm /usr/share/doc/perl-MIME-Lite-3.027/examples/mime_postcard
perl-MIME-Lite.noarch: W: spurious-executable-perm /usr/share/doc/perl-MIME-Lite-3.027/examples/mime_longlines
perl-MIME-Lite.noarch: W: spurious-executable-perm /usr/share/doc/perl-MIME-Lite-3.027/examples/nodisp
perl-MIME-Lite.noarch: W: spurious-executable-perm /usr/share/doc/perl-MIME-Lite-3.027/examples/mime_hack
perl-MIME-Lite.noarch: W: wrong-file-end-of-line-encoding /usr/share/doc/perl-MIME-Lite-3.027/README
perl-MIME-Lite.noarch: W: spurious-executable-perm /usr/share/doc/perl-MIME-Lite-3.027/examples/mime_gif
perl-MIME-Lite.noarch: W: wrong-file-end-of-line-encoding /usr/share/doc/perl-MIME-Lite-3.027/COPYING
perl-MIME-Lite.noarch: W: wrong-file-end-of-line-encoding /usr/share/doc/perl-MIME-Lite-3.027/contrib/MailTool.pm
perl-MIME-Lite.noarch: W: doc-file-dependency /usr/share/doc/perl-MIME-Lite-3.027/contrib/MailTool.pm perl(Net::Config)
perl-MIME-Lite.noarch: W: doc-file-dependency /usr/share/doc/perl-MIME-Lite-3.027/examples/mime_fwd /usr/bin/perl
perl-MIME-Lite.noarch: W: doc-file-dependency /usr/share/doc/perl-MIME-Lite-3.027/examples/mime_postcard /usr/bin/perl
perl-MIME-Lite.noarch: W: doc-file-dependency /usr/share/doc/perl-MIME-Lite-3.027/examples/mime_longlines /usr/bin/perl
perl-MIME-Lite.noarch: W: doc-file-dependency /usr/share/doc/perl-MIME-Lite-3.027/examples/nodisp /usr/bin/perl
perl-MIME-Lite.noarch: W: doc-file-dependency /usr/share/doc/perl-MIME-Lite-3.027/examples/mime_hack /usr/bin/perl
perl-MIME-Lite.noarch: W: doc-file-dependency /usr/share/doc/perl-MIME-Lite-3.027/examples/mime_gif /usr/bin/perl
perl-MIME-Lite.noarch: W: doc-file-dependency /usr/share/doc/perl-MIME-Lite-3.027/contrib/MailTool.pm perl(Mail::Address)
perl-MIME-Lite.noarch: W: doc-file-dependency /usr/share/doc/perl-MIME-Lite-3.027/examples/mime_fwd perl(Getopt::Std)
perl-MIME-Lite.noarch: W: doc-file-dependency /usr/share/doc/perl-MIME-Lite-3.027/examples/mime_longlines perl(Getopt::Std)
perl-MIME-Lite.noarch: W: doc-file-dependency /usr/share/doc/perl-MIME-Lite-3.027/examples/mime_hack perl(Getopt::Std)
perl-MIME-Lite.noarch: W: doc-file-dependency /usr/share/doc/perl-MIME-Lite-3.027/examples/mime_gif perl(Getopt::Std)
perl-MIME-Lite.noarch: W: doc-file-dependency /usr/share/doc/perl-MIME-Lite-3.027/contrib/MailTool.pm perl(Sys::Hostname)
perl-MIME-Lite.noarch: W: doc-file-dependency /usr/share/doc/perl-MIME-Lite-3.027/contrib/MailTool.pm perl(Net::SMTP)
The rest of warning is problem with examples in docs. They need interpreter, but they are not executable, so it shouldn't be a problem for review.

Comment 3 Jaroslav Škarvada 2010-01-27 14:35:59 UTC
perl-MIME-Lite-3.027-3, fedora‑review+


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