[Xen-devel] [PATCH] docs: report if we do not build a doc due to lack of the necessary tool

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

Previously only some targets did this. An alternative would be to make a hard
dependency on these tools, this might make more sense especially for markdown?

Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
---
 docs/Makefile |    9 ++++++---
 1 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/docs/Makefile b/docs/Makefile
index 2054541..3b2aaa3 100644
--- a/docs/Makefile
+++ b/docs/Makefile
@@ -58,7 +58,8 @@ python-dev-docs:
 .PHONY: man-pages
 man-pages:
        @if which $(POD2MAN) 1>/dev/null 2>/dev/null; then \
-       $(MAKE) $(DOC_MAN1) $(DOC_MAN5); fi
+       $(MAKE) $(DOC_MAN1) $(DOC_MAN5); else              \
+       echo "pod2man not installed; skipping man-pages."; fi
 
 man1/%.1: man/%.pod.1 Makefile
        $(INSTALL_DIR) $(@D)
@@ -120,14 +121,16 @@ html/%/index.html: src/%.tex
         echo "Running latex2html to generate $*/index.html ... "; \
        $(LATEX2HTML) -split 0 -show_section_numbers -toc_depth 3 -nonavigation 
\
        -numbered_footnotes -local_icons -noinfo -math -dir $(@D) \
-       $< 1>/dev/null 2>/dev/null ;fi
+       $< 1>/dev/null 2>/dev/null ; else \
+       echo "latex2html not installed; skipping $*."; fi
 
 html/%.html: %.markdown
        @$(INSTALL_DIR) $(@D)
        @set -e ; if which $(MARKDOWN) 1>/dev/null 2>/dev/null; then \
        echo "Running markdown to generate $*.html ... "; \
        $(MARKDOWN) $< > $@.tmp ; \
-       $(call move-if-changed,$@.tmp,$@) ; fi
+       $(call move-if-changed,$@.tmp,$@) ; else \
+       echo "markdown not installed; skipping $*.html."; fi
 
 txt/%.txt: %.txt
        $(INSTALL_DIR) $(@D)
-- 
1.7.7.1


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel