[PATCH] fix fdi DTD and validate fdi's

Richard Hughes hughsient at gmail.com
Mon Dec 12 14:51:19 PST 2005


On Mon, 2005-12-12 at 22:01 +0000, Richard Hughes wrote:
> On Mon, 2005-12-12 at 11:11 -0500, David Zeuthen wrote:
> > On Thu, 2005-12-08 at 20:25 -0800, Artem Kachitchkine wrote:
> > > Hi,
> > > 
> > > I tried to validate fdi files using fdi.dtd and got a whole lot of 
> > > errors. Looks like DTD is out of date. This patch updates the DTD and 
> > > adds fdi validation to 'make check'. I'm not an XML expert, please 
> > > review carefully.
> > 
> > I've committed this, thanks! Btw, only the fdi files in {information,
> > policy, preprobe} are currently used - the others only exist because we
> > can't delete directories in CVS.
> 
> I get this on make distcheck:
> 
> xmllint --noout --dtdvalid ../../../../fdi/fdi.dtd
> 10-usb-card-readers.fdi 10-usb-music-players.fdi 10-usb-zip-drives.fdi
> 10-wireless-mice.fdi 10-camera-ptp.fdi 10-usb-pda.fdi
> 10-cd-dvd-burner.fdi
> warning: failed to load external entity "10-usb-card-readers.fdi"

Trivial fix attached. Okay to apply?

Richard.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: hal-fix-xml.patch
Type: text/x-patch
Size: 631 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/hal/attachments/20051212/933aabb7/hal-fix-xml.bin


More information about the hal mailing list