Bug 166320 - Review request: perl-HTML-Format - HTML::Format Perl module
Summary: Review request: perl-HTML-Format - HTML::Format Perl module
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Ville Skyttä
QA Contact: David Lawrence
URL: http://search.cpan.org/dist/HTML-Format/
Whiteboard:
Depends On: 166318
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2005-08-19 05:43 UTC by Ralf Corsepius
Modified: 2010-07-16 17:22 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2005-08-31 15:58:58 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Ralf Corsepius 2005-08-19 05:43:26 UTC
Spec Name or Url:
ftp://packman.iu-bremen.de/fedora/SRPMS/perl-HTML-Format.spec

SRPM Name or Url:
ftp://packman.iu-bremen.de/fedora/SRPMS/perl-HTML-Format-2.04-1.src.rpm

Description:
A collection of modules that formats HTML as plaintext, PostScript or RTF.

Comment 1 Ville Skyttä 2005-08-25 18:24:11 UTC
Suggested improved Summary: HTML formatter modules for Perl 
 
 
IMO perl-Font-AFM should be Required, not only BuildRequired. 
HTML::FormatPS doesn't work without it, see setfont() there; that's not an 
optional dependency but a hard error if the needed Font::Metrics::$something 
module cannot be found.  And the code to do that is obviously so funky that 
rpm cannot detect the dependency. 
 
In my local package, I used "Requires: perl-Font-AFM" instead of "Requires: 
perl(Font::AFM)" and ditto for BuildRequires.  That's because Font::AFM is 
never actually used except for the version check in Makefile.PL.  The actually 
needed modules are Font::Metrics::*.  This is just a note, not a blocker, you 
decide what to do with it. 
 
But the suggestion to pull perl-Font-AFM in one way or another when this 
package is installed still holds. 

Comment 2 Ralf Corsepius 2005-08-26 12:06:19 UTC
You are right, updated srpm with your comments reflected to appear soon at:
ftp://packman.iu-bremen.de/fedora/SRPMS/perl-HTML-Format.spec
ftp://packman.iu-bremen.de/fedora/SRPMS/perl-HTML-Format-2.04-2.src.rpm

Comment 3 Ville Skyttä 2005-08-26 18:40:30 UTC
Approved, but please see the the suggested summary improvement from comment 1 
post-CVS-import in case you missed it. 

Comment 4 Mark Chappell 2010-07-15 15:22:01 UTC
Package Change Request
======================
Package Name: perl-HTML-Format
New Branches: EL-6
Owners: tremble


No response from owner, and it's going to be tied to EL-6 SRPMs anyway...
https://bugzilla.redhat.com/show_bug.cgi?id=612419

Comment 5 Kevin Fenzi 2010-07-16 17:22:46 UTC
spot seems to already own a EL-6 branch here.


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