Makefile: Fix Sphinx documentation builds for in-tree builds
The Sphinx build-sphinx tool does not permit building a manual into the same directory as its source files. This meant that commit5f71eac06e
broke QEMU in-source-tree builds, which would fail with: Error: source directory and destination directory are same. Fix this by making in-tree builds build the Sphinx manuals into a subdirectory of docs/. Fixes:5f71eac06e
("Makefile, configure: Support building rST documentation") Reported-by: Philippe Mathieu-Daudé <philmd@redhat.com> Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com> Message-id: 20190308135744.6480-2-peter.maydell@linaro.org
This commit is contained in:
parent
e2a18635a4
commit
1290e6711f
30
Makefile
30
Makefile
@ -655,12 +655,20 @@ dist: qemu-$(VERSION).tar.bz2
|
|||||||
qemu-%.tar.bz2:
|
qemu-%.tar.bz2:
|
||||||
$(SRC_PATH)/scripts/make-release "$(SRC_PATH)" "$(patsubst qemu-%.tar.bz2,%,$@)"
|
$(SRC_PATH)/scripts/make-release "$(SRC_PATH)" "$(patsubst qemu-%.tar.bz2,%,$@)"
|
||||||
|
|
||||||
# Note that these commands assume that there are no HTML files in
|
# Sphinx does not allow building manuals into the same directory as
|
||||||
# the docs subdir in the source tree! If there are then this will
|
# the source files, so if we're doing an in-tree QEMU build we must
|
||||||
# blow them away for an in-source-tree 'make clean'.
|
# build the manuals into a subdirectory (and then install them from
|
||||||
|
# there for 'make install'). For an out-of-tree build we can just
|
||||||
|
# use the docs/ subdirectory in the build tree as normal.
|
||||||
|
ifeq ($(realpath $(SRC_PATH)),$(realpath .))
|
||||||
|
MANUAL_BUILDDIR := docs/built
|
||||||
|
else
|
||||||
|
MANUAL_BUILDDIR := docs
|
||||||
|
endif
|
||||||
|
|
||||||
define clean-manual =
|
define clean-manual =
|
||||||
rm -rf docs/$1/_static
|
rm -rf $(MANUAL_BUILDDIR)/$1/_static
|
||||||
rm -f docs/$1/objects.inv docs/$1/searchindex.js docs/$1/*.html
|
rm -f $(MANUAL_BUILDDIR)/$1/objects.inv $(MANUAL_BUILDDIR)/$1/searchindex.js $(MANUAL_BUILDDIR)/$1/*.html
|
||||||
endef
|
endef
|
||||||
|
|
||||||
distclean: clean
|
distclean: clean
|
||||||
@ -720,8 +728,8 @@ BLOBS=
|
|||||||
endif
|
endif
|
||||||
|
|
||||||
define install-manual =
|
define install-manual =
|
||||||
for d in $$(cd docs && find $1 -type d); do $(INSTALL_DIR) "$(DESTDIR)$(qemu_docdir)/$$d"; done
|
for d in $$(cd $(MANUAL_BUILDDIR) && find $1 -type d); do $(INSTALL_DIR) "$(DESTDIR)$(qemu_docdir)/$$d"; done
|
||||||
for f in $$(cd docs && find $1 -type f); do $(INSTALL_DATA) "docs/$$f" "$(DESTDIR)$(qemu_docdir)/$$f"; done
|
for f in $$(cd $(MANUAL_BUILDDIR) && find $1 -type f); do $(INSTALL_DATA) "$(MANUAL_BUILDDIR)/$$f" "$(DESTDIR)$(qemu_docdir)/$$f"; done
|
||||||
endef
|
endef
|
||||||
|
|
||||||
# Note that we deliberately do not install the "devel" manual: it is
|
# Note that we deliberately do not install the "devel" manual: it is
|
||||||
@ -885,17 +893,17 @@ docs/version.texi: $(SRC_PATH)/VERSION
|
|||||||
# and handles "don't rebuild things unless necessary" itself.
|
# and handles "don't rebuild things unless necessary" itself.
|
||||||
# The '.doctrees' files are cached information to speed this up.
|
# The '.doctrees' files are cached information to speed this up.
|
||||||
.PHONY: sphinxdocs
|
.PHONY: sphinxdocs
|
||||||
sphinxdocs: docs/devel/index.html docs/interop/index.html
|
sphinxdocs: $(MANUAL_BUILDDIR)/devel/index.html $(MANUAL_BUILDDIR)/interop/index.html
|
||||||
|
|
||||||
# Canned command to build a single manual
|
# Canned command to build a single manual
|
||||||
build-manual = $(call quiet-command,sphinx-build $(if $(V),,-q) -b html -D version=$(VERSION) -D release="$(FULL_VERSION)" -d .doctrees/$1 $(SRC_PATH)/docs/$1 docs/$1 ,"SPHINX","docs/$1")
|
build-manual = $(call quiet-command,sphinx-build $(if $(V),,-q) -b html -D version=$(VERSION) -D release="$(FULL_VERSION)" -d .doctrees/$1 $(SRC_PATH)/docs/$1 $(MANUAL_BUILDDIR)/$1 ,"SPHINX","$(MANUAL_BUILDDIR)/$1")
|
||||||
# We assume all RST files in the manual's directory are used in it
|
# We assume all RST files in the manual's directory are used in it
|
||||||
manual-deps = $(wildcard $(SRC_PATH)/docs/$1/*.rst) $(SRC_PATH)/docs/$1/conf.py $(SRC_PATH)/docs/conf.py
|
manual-deps = $(wildcard $(SRC_PATH)/docs/$1/*.rst) $(SRC_PATH)/docs/$1/conf.py $(SRC_PATH)/docs/conf.py
|
||||||
|
|
||||||
docs/devel/index.html: $(call manual-deps,devel)
|
$(MANUAL_BUILDDIR)/devel/index.html: $(call manual-deps,devel)
|
||||||
$(call build-manual,devel)
|
$(call build-manual,devel)
|
||||||
|
|
||||||
docs/interop/index.html: $(call manual-deps,interop)
|
$(MANUAL_BUILDDIR)/interop/index.html: $(call manual-deps,interop)
|
||||||
$(call build-manual,interop)
|
$(call build-manual,interop)
|
||||||
|
|
||||||
qemu-options.texi: $(SRC_PATH)/qemu-options.hx $(SRC_PATH)/scripts/hxtool
|
qemu-options.texi: $(SRC_PATH)/qemu-options.hx $(SRC_PATH)/scripts/hxtool
|
||||||
|
Loading…
Reference in New Issue
Block a user