From 6d16f234b8344b50c6b513e4969d2e87ab16dc08 Mon Sep 17 00:00:00 2001 From: Quentin Young Date: Tue, 13 Mar 2018 13:18:19 -0400 Subject: [PATCH] doc: don't check for sphinx-build on `make clean` Signed-off-by: Quentin Young --- doc/frr-sphinx.mk | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/doc/frr-sphinx.mk b/doc/frr-sphinx.mk index 3d91e8f63d..df4056760d 100644 --- a/doc/frr-sphinx.mk +++ b/doc/frr-sphinx.mk @@ -11,11 +11,13 @@ PAPER ?= BUILDDIR = _build # User-friendly check for sphinx-build -ifeq ($(shell which $(SPHINXBUILD) >/dev/null 2>&1; echo $$?), 1) -SPHINXBUILD = sphinx-1.0-build -endif -ifeq ($(shell which $(SPHINXBUILD) >/dev/null 2>&1; echo $$?), 1) -$(error "The '$(SPHINXBUILD)' command was not found. Make sure you have Sphinx installed, then set the SPHINXBUILD make variable to point to the full path of the '$(SPHINXBUILD)' executable. Alternatively you can add the directory with the executable to your PATH. If you don't have Sphinx installed, grab it from http://sphinx-doc.org/") +ifneq ($(MAKECMDGOALS), clean) + ifeq ($(shell which $(SPHINXBUILD) >/dev/null 2>&1; echo $$?), 1) + SPHINXBUILD = sphinx-1.0-build + endif + ifeq ($(shell which $(SPHINXBUILD) >/dev/null 2>&1; echo $$?), 1) + $(error "The '$(SPHINXBUILD)' command was not found. Make sure you have Sphinx installed, then set the SPHINXBUILD make variable to point to the full path of the '$(SPHINXBUILD)' executable. Alternatively you can add the directory with the executable to your PATH. If you don't have Sphinx installed, grab it from http://sphinx-doc.org/") + endif endif # Internal variables. -- 2.39.5