[Bug 166318] Review request: perl-Font-AFM - Font::AFM Perl module

bugzilla at redhat.com bugzilla at redhat.com
Sat Aug 20 09:15:20 UTC 2005


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review request: perl-Font-AFM - Font::AFM Perl module


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=166318


ville.skytta at iki.fi changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|163778                      |163779
              nThis|                            |




------- Additional Comments From ville.skytta at iki.fi  2005-08-20 05:15 EST -------
I checked freebsd and gentoo too, and they don't modify it either. 
 
So it boils down to whether the default value in the unmodified AFM.pm is 
useful on the majority of OS's; if not, applications using it will have to 
take it into account anyway some way, and thus having a better default value 
here won't help much.  On the other hand, then it might be that instead of 
having a better defaults here, there could be several applications using 
Font::AFM that would need modification, which is probably even worse regarding 
maintenance. 
 
Also, the *.afm filenames seem to vary pretty much between the packages that 
provide them, like the test suite here expecting to find Helvetica.afm, which 
is shipped at least with the name Helvetica.afm in some OpenOffice's dirs, 
phvr.afm in a2ps and hv.afm in enscript. 
 
So, pick your poison, I don't see the default paths being a blocker, modified 
or unmodified, so consider this approved.  The test suite coverage and summary 
improvement bits should be committed, though. 

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.




More information about the fedora-extras-list mailing list