From 4c36d45a2cc03ce76307c1183831e622e7325e26 Mon Sep 17 00:00:00 2001 From: Dan Willemsen Date: Mon, 26 Feb 2018 18:26:44 -0800 Subject: [PATCH] Use xmllint built from source Instead of relying on whatever version of xmllint is on the host system, build and use the version in external/libxml2. Test: diff build_aosp-marlin.ninja, expected changes. Test: m $(xmllint targets in build_aosp-marlin.ninja) Test: introduce xml error, build fails Change-Id: I39579f06db3777e3b5c8dda7c7541c25a35887b2 --- core/config.mk | 1 + core/definitions.mk | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/core/config.mk b/core/config.mk index 4942be73c..295644c40 100644 --- a/core/config.mk +++ b/core/config.mk @@ -580,6 +580,7 @@ MAKEPARALLEL := $(prebuilt_build_tools_bin)/makeparallel SOONG_JAVAC_WRAPPER := $(SOONG_HOST_OUT_EXECUTABLES)/soong_javac_wrapper SOONG_ZIP := $(SOONG_HOST_OUT_EXECUTABLES)/soong_zip MERGE_ZIPS := $(SOONG_HOST_OUT_EXECUTABLES)/merge_zips +XMLLINT := $(SOONG_HOST_OUT_EXECUTABLES)/xmllint ZIP2ZIP := $(SOONG_HOST_OUT_EXECUTABLES)/zip2zip ZIPTIME := $(prebuilt_build_tools_bin)/ziptime diff --git a/core/definitions.mk b/core/definitions.mk index 2f3f35655..f6f5840fb 100644 --- a/core/definitions.mk +++ b/core/definitions.mk @@ -2686,9 +2686,9 @@ endef # $(1): source file # $(2): destination file, must end with .xml. define copy-xml-file-checked -$(2): $(1) +$(2): $(1) $(XMLLINT) @echo "Copy xml: $$@" - $(hide) xmllint $$< >/dev/null # Don't print the xml file to stdout. + $(hide) $(XMLLINT) $$< >/dev/null # Don't print the xml file to stdout. $$(copy-file-to-target) endef