Bug 166320

Summary: Review request: perl-HTML-Format - HTML::Format Perl module
Product: [Fedora] Fedora Reporter: Ralf Corsepius <rc040203>
Component: Package ReviewAssignee: Ville Skyttä <scop>
Status: CLOSED NEXTRELEASE QA Contact: David Lawrence <dkl>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-extras-list, tremble
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
URL: http://search.cpan.org/dist/HTML-Format/
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2005-08-31 15:58:58 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: 166318    
Bug Blocks: 163779    

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.