$(DOC)/%.html: %.sgml
        echo Creating html for $< to $@
        -rm -rf $@
-       (HERE=`pwd`; cd $(@D) && $(DEBIANDOC_HTML) $(DEBIANDOC_HTML_OPTIONS) $$HERE/$<)
+       (HERE=`pwd`; cd $(@D) && $(DEBIANDOC_HTML) $(DEBIANDOC_HTML_OPTIONS) $$HERE/$<) || exit 199
 
 # Clean rule
 .PHONY: veryclean/html/$(LOCAL)
 vpath %.sgml $(SUBDIRS)
 $(DOC)/%.text: %.sgml
        echo Creating text for $< to $@
-       $(DEBIANDOC_TEXT) -O $< > $@
+       $(DEBIANDOC_TEXT) -O $< > $@ || exit 198
 
 # Clean rule
 .PHONY: veryclean/text/$(LOCAL)
 
 
 $($(LOCAL)-LIST) :: % : %.xml $(INCLUDES)
        echo Creating man page $@
-       $(XSLTPROC) -o $@ $(STYLESHEET) $< # why xsltproc doesn't respect the -o flag here???
+       $(XSLTPROC) -o $@ $(STYLESHEET) $< || exit 200 # why xsltproc doesn't respect the -o flag here???
        test -f $(subst .$(LC),,$@) || echo FIXME: xsltproc respect the -o flag now, workaround can be removed
        mv -f $(subst .$(LC),,$@) $@
 
 
   * debian/control:
     - build-depend on docbook-xml to ensure that the xml DTDs are always
       available on the buildds (Closes: #597145)
+  * buildlib/debiandoc.mak, buildlib/po4a_manpage.mak:
+    - ensure that the build fails if documentation building fails
 
- -- David Kalnischkies <kalnischkies@gmail.com>  Fri, 17 Sep 2010 09:53:04 +0200
+ -- David Kalnischkies <kalnischkies@gmail.com>  Fri, 17 Sep 2010 10:01:45 +0200
 
 apt (0.8.4) unstable; urgency=low