Bug 477551 - RFE Add fontconfig file checking
Summary: RFE Add fontconfig file checking
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: rpmlint
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Ville Skyttä
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-12-21 21:53 UTC by Nicolas Mailhot
Modified: 2008-12-22 14:43 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-12-22 13:54:35 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Nicolas Mailhot 2008-12-21 21:53:56 UTC
As suggested by Jens Petersen the package review tools such as rpm should be enhanced to check fontconfig files provided by packagers are well-formed.

This check could take two forms:
1. check that all the files in %_fontconfig_templatedir are well-formed-XML
2. check that they conform to the current fontconfig DTD (this test probably requires for the fontconfig packager to register it in the system XML catalogs but Behdad is working on a new release right now so that's a good time to ask)

Comment 1 Ville Skyttä 2008-12-22 13:54:35 UTC
From rpmlint POV, I think a generic XML well-formedness/validation check would be a better approach than doing this just for fontconfig files.  I've filed an upstream RFE about this: http://rpmlint.zarb.org/cgi-bin/trac.cgi/ticket/168

In the meantime (and even if/when implemented in rpmlint), I suggest font packages could add a %check section and validate their XML files in it - it should be easy for them to do it using xmllint even if the needed DTD is not catalogued (obviously as long as it is available somewhere locally).

Comment 2 Nicolas Mailhot 2008-12-22 14:10:19 UTC
Well, the problem with fontconfig is that its XML files do not use the xml extension, so you'll need to special-case their detection in any case.

Comment 3 Ville Skyttä 2008-12-22 14:43:25 UTC
Maybe ask fontconfig developers if they'd be willing to change the extension (if they're XML, why not *.xml)?  Anyway, checking other stuff in rpmlint than just *.xml would certainly be doable.  Added some related comments to upstream RFE, please add further comments there.


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