[PATCH] fix fdi DTD and validate fdi's

Richard Hughes hughsient at gmail.com
Mon Dec 12 14:01:48 PST 2005


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"
warning: failed to load external entity "10-usb-music-players.fdi"
warning: failed to load external entity "10-usb-zip-drives.fdi"
warning: failed to load external entity "10-wireless-mice.fdi"
warning: failed to load external entity "10-camera-ptp.fdi"
warning: failed to load external entity "10-usb-pda.fdi"
warning: failed to load external entity "10-cd-dvd-burner.fdi"
make[4]: *** [check] Error 1
make[4]: Leaving directory
`/home/hughsie/hal-devel/cvs/hal-vanilla/hal-0.5.6/_build/fdi/information/10freedesktop'
make[3]: *** [check-recursive] Error 1
make[3]: Leaving directory
`/home/hughsie/hal-devel/cvs/hal-vanilla/hal-0.5.6/_build/fdi/information'
make[2]: *** [check-recursive] Error 1
make[2]: Leaving directory
`/home/hughsie/hal-devel/cvs/hal-vanilla/hal-0.5.6/_build/fdi'
make[1]: *** [check-recursive] Error 1
make[1]: Leaving directory
`/home/hughsie/hal-devel/cvs/hal-vanilla/hal-0.5.6/_build'
make: *** [distcheck] Error 2

Richard.



More information about the hal mailing list