[Xen-devel] [PATCH for-4.9] build: more adjustments to top-level Makefile dependencies

[Wei Liu <wei.liu2@xxxxxxxxxx>]

In the original code, top-level dist target unconditionally invokes
dist target for tools/include, which is wrong when tools component is
not enabled.

Make dist-tools depend on dist-tools-public-headers, which depends on
build-tools-public-headers.

Discovered by Travis-CI.

Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx>
---
Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
Cc: Julien Grall <Julien.Grall@xxxxxxx>

Tested with

1. ./configure --disable-tools --disable-stubdom then make -j8 dist
2. ./configure, make -j8 dist-stubdom then make -j8 dist-tools
3. ./configure --disable-tools then make -j8 dist
3. ./configure, make -j8 dist

Travis is now happy, too.

I'm sure make --enable-stubdom --disable-tools didn't work with or
without this patch, because tools/include needs files generated by
tools' ./configure.
---
 Makefile | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index e958bdc44b..fc30b3cc29 100644
--- a/Makefile
+++ b/Makefile
@@ -42,6 +42,10 @@ export DESTDIR
 build-tools-public-headers:
        $(MAKE) -C tools/include
 
+.PHONY: dist-tools-public-headers
+dist-tools-public-headers: build-tools-public-headers
+       $(MAKE) -C tools/include dist
+
 # build and install everything into the standard system directories
 .PHONY: install
 install: $(TARGS_INSTALL)
@@ -80,13 +84,15 @@ test:
 .PHONY: dist
 dist: DESTDIR=$(DISTDIR)/install
 dist: $(TARGS_DIST) dist-misc
-       make -C tools/include dist
 
 dist-misc:
        $(INSTALL_DIR) $(DISTDIR)/
        $(INSTALL_DATA) ./COPYING $(DISTDIR)
        $(INSTALL_DATA) ./README $(DISTDIR)
        $(INSTALL_PROG) ./install.sh $(DISTDIR)
+
+
+dist-tools: dist-tools-public-headers
 dist-%: DESTDIR=$(DISTDIR)/install
 dist-%: install-%
        @: # do nothing
-- 
2.11.0


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel