Re: [libvirt] [PATCH] Let make fail when XHTML validation fails

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> > diff --git a/docs/Makefile.am b/docs/Makefile.am
> > index eaac627..4df5154 100644
> > --- a/docs/Makefile.am
> > +++ b/docs/Makefile.am
> > @@ -108,7 +108,7 @@ ChangeLog.html.in: ChangeLog.xml ChangeLog.xsl
> >  	@(if [ -x $(XMLLINT) -a -x $(XMLCATALOG) ] ; then \
> >  	  if $(XMLCATALOG) /etc/xml/catalog "-//W3C//DTD XHTML 1.0 Strict//EN" > /dev/null ; then \
> >  	  echo "Validating $@" ; \
> > -	  $(XMLLINT) --nonet --format --valid $< > $@ || : ; \
> > +	  $(XMLLINT) --nonet --format --valid $< > $@ || (rm $@ && exit 1) ; \
> >  	  else echo "missing XHTML1 DTD" ; fi ; fi );
> 
>   Hum, maybe that's a bit excessive, while it make sense to see the
> error in a development environment, I find this a bit hard at
> installation time.

It shouldn't cause any troubles at installation time. If either xmllint,
xmlcatalog or XHTML DTDs are not found, the validation is just skipped
without any error. If all of those are found, there is no reason why it would
fail as all html.in files which are in git are valid XHTML 1.0. Or did I miss
anything?

Jirka

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]