Skip to content
Permalink
Browse files

don't require antora

if we're not building "docsite", we don't care if "antora"
is missing.

And "docsite" contains the string "doc", so we don't need to list
them both.
  • Loading branch information
alandekok committed Aug 17, 2019
1 parent 107c8e6 commit fa8119f94485f6f77e8269783b8729ec6e255eb2
Showing with 4 additions and 1 deletion.
  1. +4 −1 doc/all.mk
@@ -6,7 +6,7 @@
# to fix that. So, only run those shell scripts if we're going to
# build the documentation.
#
WITH_DOC := $(strip $(foreach x,doc html docsite pdf doxygen,$(findstring $(x),$(MAKECMDGOALS))))
WITH_DOC := $(strip $(foreach x,doc html pdf doxygen,$(findstring $(x),$(MAKECMDGOALS))))
ifneq "$(WITH_DOC)" ""

#
@@ -25,10 +25,13 @@ endif

#
# We're building a documentation target, but there's no "antora".
# Which we ONLY need for "docsite"
#
ifeq "$(ANTORA)" ""
ifneq "$(findstring docsite,$(MAKECMDGOALS))" ""
$(error antora is required to build the documentation)
endif
endif

#
# We're installing the documentation, but there's no "docdir".

0 comments on commit fa8119f

Please sign in to comment.
You can’t perform that action at this time.