Bug 219036
Summary: | Review Request: perl-Roman - Roman module from CPAN | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Sam Varshavchik <mrsam> | ||||
Component: | Package Review | Assignee: | Jose Pedro Oliveira <jose.p.oliveira.oss> | ||||
Status: | CLOSED NOTABUG | QA Contact: | Fedora Package Reviews List <fedora-package-review> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | rawhide | ||||||
Target Milestone: | --- | ||||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2007-06-02 03:29:00 UTC | Type: | --- | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Bug Depends On: | |||||||
Bug Blocks: | 201449 | ||||||
Attachments: |
|
Description
Sam Varshavchik
2006-12-09 18:44:35 UTC
NEEDSWORK MD5SUMS: 3bcd42265a01a9261b18146273872a0d perl-Roman-1.1-0.fc.1.src.rpm 0ded0e59e44267c803c4c8609ee52523 perl-Roman.spec 67b0e6affdc50fdf28cfc438c045fd9b Roman-1.1.tar.gz Sources: * Roman-1.1.tar.gz: MD5 digest matches the CPAN tarball NeedsWork: * wrong release (and dist tag) * the license information is incomplete: perl is dual licensed (GPL or Artistic) * changelog entry missing * remove the dot at the end of the summary (rpmlint) Suggestions: * change the URL to "http://search.cpan.org/dist/Roman/" (no author or version info - easier to maintain) * the perl build requirement can be dropped * when copying files try to preserve the timestamps (use install with the -p option) * you can also use install to create the perl directory (%{__install} -d -m 0755 $RPM_BUILD_ROOT%{perl_vendorlib}) * don't mix spaces and tabs in the specfile (check the release line) Created attachment 143242 [details]
specfile patch
Sam,
The attached patch corrects all the problems mentioned in the previous comment
(also removes a couple of lost spaces and reformats a couple of lines).
jpo
Thanks for doing my work for me. I'll slap myself later, for forgetting to use rpmlint. Updated spec file: http://www.courier-mta.com/perl-Roman.spec Updated SRPM: http://www.courier-mta.com/perl-Roman-1.1-1.src.rpm APPROVED MD5SUMS: df12af9eef0433eb70f89c431111c05e perl-Roman-1.1-1.src.rpm 7c93de8cd3de204072b194a561368629 perl-Roman.spec 67b0e6affdc50fdf28cfc438c045fd9b Roman-1.1.tar.gz Sources: * Roman-1.1.tar.gz: MD5 digest matches the CPAN tarball Good: * Package name follows standard * URL and Source url are OK * License verified (main pod page and License file) * perl(:MODULE_COMPAT_xxx) requirement present * the requirements are sane * the provides are sane * perl vendor libs macros present * File permissions are ok * Builds without problems in FC-6 * (Un)installs without problems in FC-6 * Runs on FC-6 (perl -MRoman -e 'print Roman(1004);') * No opened tickets in http://rt.cpan.org/Public/Dist/Display.html?Name=Roman Random notes about this module: * no Makefile.PL/Build.PL * no testsuite * no VERSION variable jpo ping! Was this ping directed at me? Is there something else that I need to do? Sam, (In reply to comment #6) > Was this ping directed at me? Yes. > Is there something else that I need to do? Please read http://fedoraproject.org/wiki/Extras/Contributors and, if this is your first package, see also http://fedoraproject.org/wiki/Extras/HowToGetSponsored and add FE-NEEDSPONSOR to the blocking list (see step 7 of the first link). Regards, jpo Anything happening here? I guess not; I'll close this bug soon if there is no response. |