Bug 166190
Summary: | Review Request: perl-Class-DBI-Pager : Pager utility for Class::DBI | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Tom "spot" Callaway <tcallawa> | ||||
Component: | Package Review | Assignee: | Paul Howarth <paul> | ||||
Status: | CLOSED NEXTRELEASE | QA Contact: | David Lawrence <dkl> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | rawhide | CC: | fedora-package-review | ||||
Target Milestone: | --- | ||||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
URL: | http://search.cpan.org/dist/Class-DBI-Pager/ | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2005-09-08 15:38:21 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: | 166184, 166199 | ||||||
Bug Blocks: | 163779, 166203 | ||||||
Attachments: |
|
Description
Tom "spot" Callaway
2005-08-17 20:33:57 UTC
Review: - rpmlint clean - package and spec naming OK - package meets guidelines - license is same as perl, matches spec - spec file written in ENglish and is legible - sources match upstream - builds OK in mock on FC4 (i386) - no locales, libraries, subpackages or pkgconfigs to worry about - not relocatable - do 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 scriptlets Needswork: - remove redundant BR: perl - add BR: perl(Data::Page), perl(Exporter::Lite), perl(DBD::SQLite) - include license text Nitpicks: - honor %%{_smp_mflags} - remove explicit perl(Class::DBI) dep - it's autodetected Created attachment 118457 [details]
Patch addressing review issues
fixed in -3: SRPM: http://www.auroralinux.org/people/spot/review/Maypole/perl-Class-DBI-Pager-0.06-3.src.rpm SPEC: http://www.auroralinux.org/people/spot/review/Maypole/perl-Class-DBI-Pager.spec Approved. |