[libvirt] [PATCHv2 3/2] docs/Makefile.am: remove unnecessary subshells

Eric Blake eblake at redhat.com
Thu Apr 29 01:37:03 UTC 2010


* docs/Makefile.am (ChangeLog.html.in, %.html.tmp, %.html)
(html/index.html, $(devhelphtml)): Avoid spurious subshells.
---

This is pretty trivial; I noticed it while building the previous patch.

 docs/Makefile.am |   32 +++++++++++++++++---------------
 1 files changed, 17 insertions(+), 15 deletions(-)

diff --git a/docs/Makefile.am b/docs/Makefile.am
index 91ae13e..634d456 100644
--- a/docs/Makefile.am
+++ b/docs/Makefile.am
@@ -92,49 +92,51 @@ ChangeLog.xml: ../ChangeLog ChangeLog.awk
 	awk -f ChangeLog.awk < $< > $@

 ChangeLog.html.in: ChangeLog.xml ChangeLog.xsl
-	@(if [ -x $(XSLTPROC) ] ; then \
+	@if [ -x $(XSLTPROC) ] ; then \
 	  echo "Generating $@"; \
 	  name=`echo $@ | sed -e 's/.tmp//'`; \
-	  $(XSLTPROC) --nonet $(top_srcdir)/docs/ChangeLog.xsl $< > $@ || (rm $@ && exit 1) ; fi )
+	  $(XSLTPROC) --nonet $(top_srcdir)/docs/ChangeLog.xsl $< > $@ \
+	    || { rm $@ && exit 1; }; fi

 %.png: %.fig
 	convert -rotate 90 $< $@

 %.html.tmp: $(srcdir)/%.html.in site.xsl page.xsl sitemap.html.in
-	@(if [ -x $(XSLTPROC) ] ; then \
+	@if [ -x $(XSLTPROC) ] ; then \
 	  echo "Generating $@"; \
 	  name=`echo $@ | sed -e 's/.tmp//'`; \
-	  $(XSLTPROC) --stringparam pagename $$name --nonet --html $(top_srcdir)/docs/site.xsl $< > $@ || (rm $@ && exit 1) ; fi )
+	  $(XSLTPROC) --stringparam pagename $$name --nonet --html \
+	    $(top_srcdir)/docs/site.xsl $< > $@ \
+	    || { rm $@ && exit 1; }; fi

 %.html: %.html.tmp
-	@(if test -x $(XMLLINT) && test -x $(XMLCATALOG) ; then \
+	@if test -x $(XMLLINT) && test -x $(XMLCATALOG) ; then \
 	  if $(XMLCATALOG) /etc/xml/catalog \
 	    "-//W3C//DTD XHTML 1.0 Strict//EN" > /dev/null ; then \
 	  echo "Validating $@" ; \
 	  $(XMLLINT) --nonet --format --valid $< > $@ \
-	    || (rm $@ && exit 1) ; \
-	  else echo "missing XHTML1 DTD" ; fi ; fi );
-
+	    || { rm $@ && exit 1; }; \
+	  else echo "missing XHTML1 DTD" ; fi ; fi

 $(srcdir)/html/index.html: $(srcdir)/libvirt-api.xml newapi.xsl page.xsl \
 		sitemap.html.in
-	-@(if [ -x $(XSLTPROC) ] ; then \
+	- at if [ -x $(XSLTPROC) ] ; then \
 	  echo "Rebuilding the HTML pages from the XML API" ; \
 	  $(XSLTPROC) --nonet $(srcdir)/newapi.xsl $(srcdir)/libvirt-api.xml; \
-	fi )
-	-@(if test -x $(XMLLINT) && test -x $(XMLCATALOG) ; then \
+	fi
+	- at if test -x $(XMLLINT) && test -x $(XMLCATALOG) ; then \
 	  if $(XMLCATALOG) /etc/xml/catalog "-//W3C//DTD XHTML 1.0 Strict//EN" \
 	    > /dev/null ; then \
 	  echo "Validating the resulting XHTML pages" ; \
 	  $(XMLLINT) --nonet --valid --noout html/*.html ; \
-	  else echo "missing XHTML1 DTD" ; fi ; fi );
+	  else echo "missing XHTML1 DTD" ; fi ; fi

 $(devhelphtml): $(srcdir)/libvirt-api.xml $(devhelpxsl)
-	-@(echo Rebuilding devhelp files)
-	-@(if [ -x $(XSLTPROC) ] ; then \
+	- at echo Rebuilding devhelp files
+	- at if [ -x $(XSLTPROC) ] ; then \
 	  $(XSLTPROC) --nonet -o devhelp/libvirt.devhelp \
 	  $(top_srcdir)/docs/devhelp/devhelp.xsl $(srcdir)/libvirt-api.xml ; \
-	fi );
+	fi

 python_generated_files = \
 		$(srcdir)/html/libvirt-libvirt.html \
-- 
1.6.6.1




More information about the libvir-list mailing list