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 214527 - Review Request: perl-HTML-Table - Create HTML tables using simple interface
Summary: Review Request: perl-HTML-Table - Create HTML tables using simple interface
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Kevin Fenzi
QA Contact: Fedora Package Reviews List
URL: http://www.cora.nwra.com/~orion/fedora/
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-11-07 23:30 UTC by Orion Poplawski
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-11-16 19:08:26 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Orion Poplawski 2006-11-07 23:30:21 UTC
Spec Name or Url: http://www.cora.nwra.com/~orion/fedora/perl-HTML-Table.spec
SRPM Name or Url:
http://www.cora.nwra.com/~orion/fedora/perl-HTML-Table-2.04a-1.fc6.src.rpm
Description: 

HTML::Table is used to generate HTML tables for CGI scripts.  By using the
methods provided fairly complex tables can be created, manipulated, then
 printed from Perl scripts.  The module also greatly simplifies creating
tables within tables from Perl.  It is possible to create an entire table
using the methods provided and never use an HTML tag.

HTML::Table also allows for creating dynamically sized tables via its addRow
and addCol methods.  These methods automatically resize the table if passed
more cell values than will fit in the current table grid.

Methods are provided for nearly all valid table, row, and cell tags specified
for HTML 3.0.

Comment 1 Kevin Fenzi 2006-11-12 03:38:46 UTC
OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name.
OK - Spec has consistant macro usage.
OK - Meets Packaging Guidelines.
OK - License
OK - License field in spec matches
See below - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
12c107450c8759e9dc0f8dea91297132  HTML-Table-2.04a.tar.gz
12c107450c8759e9dc0f8dea91297132  HTML-Table-2.04a.tar.gz.1
OK - BuildRequires correct
OK - Package has %defattr and permissions on files is good.
OK - Package has a correct %clean section.
OK - Package has correct buildroot
OK - Package is code or permissible content.
OK - Packages %doc files don't affect runtime.

OK - Package compiles and builds on at least one arch.
OK - Package has no duplicate files in %files.
OK - Package doesn't own any directories other packages own.
OK - Package owns all the directories it creates.
OK - No rpmlint output.
OK - final provides and requires are sane:

SHOULD Items:

OK - Should build in mock.
OK i386/x86_64 - Should build on all supported archs
OK - Should have dist tag
OK - Should package latest version

Issues:

1. Any chance to get upstream to include a license file?
Thats not a blocker however, and thats the only issue I see...

this package is APPROVED.

Don't forget to close this bug NEXTRELEASE once it's been imported
and built.

Also, consider reviewing another package thats waiting to help
spread out the reviewing load.


Comment 2 Orion Poplawski 2006-11-16 19:08:26 UTC
Thanks for the review.  Checked in and added to owners.list.


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