doc/doxygen: Add a check for the GraphViz dependency

crasbe · crasbe · commit 64898cf13af1 · 2025-04-04T17:59:53.000+02:00
diff --git a/doc/doxygen/Makefile b/doc/doxygen/Makefile
@@ -43,14 +43,23 @@ include $(RIOTMAKE)/utils/ansi.mk
 include $(RIOTMAKE)/color.inc.mk
 
 .PHONY: doc doc-man doc-latex
-doc doc-man doc-latex: $(DOCUMENTATION_FORMAT)
+doc doc-man doc-latex: graphviz-check $(DOCUMENTATION_FORMAT)
 	@if [ "`{ echo "$(DOXYGEN_MIN_VERSION)"; echo "$(DOXYGEN_CUR_VERSION)"; } | \
 		sort -V | head -n1`" != "$(DOXYGEN_MIN_VERSION)" ]; then \
 		$(COLOR_ECHO) "$(COLOR_RED)Warning: Doxygen version $(DOXYGEN_CUR_VERSION) is too old." \
 		"It is recommended to use at least version $(DOXYGEN_MIN_VERSION)" \
 		"to avoid incorrectly formatted output.$(COLOR_RESET)"; \
 	fi
 
+# GraphViz is a mandatory dependency to generate the Doxygen documentation for RIOT
+.PHONY: graphviz-check
+graphviz-check:
+	@command -v dot >/dev/null 2>&1 || { \
+		$(COLOR_ECHO) "$(COLOR_RED)Error: GraphViz (dot) is not installed." \
+    "GraphViz is mandatory to generate the RIOT documentation.$(COLOR_RESET)"; \
+		exit 1; \
+	}
+
 .PHONY: doc-ci
 doc-ci: $(RIOTTOOLS)/doxygen/Makefile
 	@PKG_VERSION=$(DOXYGEN_VERSION) $(MAKE) -C $(RIOTTOOLS)/doxygen all --no-print-directory
