Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 188261
Summary: | Review Request: perl-Finance-Quote | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Bill Nottingham <notting> |
Component: | Package Review | Assignee: | Paul Howarth <paul> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | rvokal, sig |
Target Milestone: | --- | Keywords: | Reopened |
Target Release: | --- | Flags: | j:
fedora-review+
j: fedora-cvs+ |
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2007-06-12 20:12:13 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: | 188293 | ||
Bug Blocks: | 163779 |
Description
Bill Nottingham
2006-04-07 14:52:18 UTC
*** Bug 161410 has been marked as a duplicate of this bug. *** Please submit a package for perl(HTML::TableExtract) too, since it is a dependency of this package. The "BuildRequires: perl >= 1:5.6.1" is redundant for Fedora Extras The "|| :" after "%check" shouldn't be needed in Extras packages. The various module files included in this package generate lots of rpmlint errors about non-executable-scripts, which could be eliminated by removing the shellbangs from the .pm files in %prep. Heh. Wonder where I got my TableExtract from. In the meantime, new stuff uploaded. perl-HTML-TableExtract added for review in bug 188293. Please bump the release number for package revisions during the review phase - it helps make clear which comments relate to which version of the package. I'm unable to extract the current SRPM: perl-Finance-Quote-1.11-1/cpio: premature end of file The file I have is 89583 bytes, so I appear to have all of what's on the server. Please ensure that perl(HTML::TableExtract) is a buildreq for this package if you haven't already done so. How is it a buildreq? I'll admit, I'm not 100% up on the perl build processes. -2 uploaded. (In reply to comment #6) > How is it a buildreq? I'll admit, I'm not 100% up on the perl build processes. Strictly speaking it isn't - yet. The Makefile.PL has it as a prerequisite and without it, you get this during the build: Checking if your kit is complete... Looks good Warning: prerequisite HTML::TableExtract 0 not found. Writing Makefile for Finance::Quote Normally that would spell imminent doom when it came to running the test suite (HTML::TableExtract would be needed in order to run any meaningful test), but in this case you get away with it because there isn't actually any test suite. Adding the buildreq now would make the package "ready" for when upstream actually provides a test suite and is just good practice anyway. The find/while/read/grep/mv loop in %prep could be simplified to: find . -name *.pm | xargs %{__sed} -i -e '/^#!.*\/usr\/bin\/perl/d' There is no changelog entry for the -2 revision. Other than that, looks good. -3 uploaded with buildreq, sed fix, and changelog. Review: - rpmlint clean - package and spec naming OK - package meets guidelines - license is GPL, matches spec, text included - spec file written in English and is legible - sources match upstream - package builds ok in mock for rawhide (i386) (with perl-HTML-TableExtract in local repo for now) - buildteqs OK - no locales, libraries, pkgconfigs, or subpackages to worry about - not relocatable - no directory ownership or permissions issues - no duplicate files - %clean section present and correct - macro usage is consistent - code, not content - no large docs - docs don't affect runtime - no .desktop file needed - module appears to retrieve stock prices correctly - no scriptlets Approved. Built. Package Change Request ---------------------- Package: perl-Finanace-Quote New Branches: EL-4 EL-5 CVS done. BTW, there's no need to reopen old tickets to make CVS requests. |