From e87ff2d85fbad942f6ed698320aed40412c83bea Mon Sep 17 00:00:00 2001 From: Logan Chien Date: Thu, 14 Feb 2019 14:27:33 +0800 Subject: [PATCH] Strip LOCAL_CHECK_ELF_FILES This commit strips LOCAL_CHECK_ELF_FILES before comparing it with `false` so that developers can add comments after LOCAL_CHECK_ELF_FILES. For example: LOCAL_CHECK_ELF_FILES := false # disable checks due to ... Bug: 119084334 Test: Add some spaces after LOCAL_CHECK_ELF_FILES and run CHECK_ELF_FILES=true make check-elf-files Change-Id: I2bfaff4807aeeea6c7af0b9d7d716742b6764cd0 --- core/check_elf_file.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/core/check_elf_file.mk b/core/check_elf_file.mk index 69d891cfd..8f2eafb11 100644 --- a/core/check_elf_file.mk +++ b/core/check_elf_file.mk @@ -35,7 +35,7 @@ $(check_elf_files_stamp): $(my_prebuilt_src_file) $(CHECK_ELF_FILE) $(LLVM_READO $(hide) touch $@ ifneq ($(PRODUCT_CHECK_ELF_FILES)$(CHECK_ELF_FILES),) -ifneq ($(LOCAL_CHECK_ELF_FILES),false) +ifneq ($(strip $(LOCAL_CHECK_ELF_FILES)),false) $(LOCAL_BUILT_MODULE): $(check_elf_files_stamp) check-elf-files: $(check_elf_files_stamp) endif # LOCAL_CHECK_ELF_FILES