From 3e87e835573d8ef36338331418b73169d0bee180 Mon Sep 17 00:00:00 2001 From: Colin Cross Date: Mon, 27 Jan 2020 13:52:28 -0800 Subject: [PATCH] Give strip depfile a unique name, properly include it $(LOCAL_BUILT_MODULE).d may conflict with the .d file for a .o file, use $(LOCAL_BUILT_MODULE).strip.d. Also pass the required second arg to include-depfile. Bug: 148387048 Test: m checkbuild Change-Id: I1f5f4c38010294a12d6777439b0f66c0102c1c96 --- core/dynamic_binary.mk | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/core/dynamic_binary.mk b/core/dynamic_binary.mk index 27ff2c969..48072b369 100644 --- a/core/dynamic_binary.mk +++ b/core/dynamic_binary.mk @@ -132,8 +132,8 @@ ifneq (,$(my_strip_module)) CLANG_BIN=$(LLVM_PREBUILTS_PATH) \ CROSS_COMPILE=$(PRIVATE_TOOLS_PREFIX) \ XZ=$(XZ) \ - $(SOONG_STRIP_PATH) -i $< -o $@ -d $@.d $(PRIVATE_STRIP_ARGS) - $(call include-depfile,$(strip_output).d) + $(SOONG_STRIP_PATH) -i $< -o $@ -d $@.strip.d $(PRIVATE_STRIP_ARGS) + $(call include-depfile,$(strip_output).strip.d,$(strip_output)) else # Don't strip the binary, just copy it. We can't skip this step # because a copy of the binary must appear at LOCAL_BUILT_MODULE.