tequilaOS/platform_build
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Remove duplicate includes of BUILD_NOTICE_FILE

Browse source 
Some module types include BUILD_NOTICE_FILE a second time after
the include from base_rules.mk.  This won't work after a future
change, and doesn't seem to be necessary as none of the files
leave any modified LOCAL* variables in place for notice_files.mk
to read.

Bug: 207445310
Test: m checkbuild
Change-Id: I8c68888d6bf26926f5c2216abe147ccf28ddc2b8
This commit is contained in:
Colin Cross 2021-11-29 21:50:45 -08:00
parent 6013329bff
commit 143af9284f
3 changed files with 0 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
core/host_java_library.mk
View file

@ -131,8 +131,3 @@ $(eval $(call copy-one-file,$(LOCAL_FULL_CLASSES_JACOCO_JAR),$(full_classes_jar)
ifeq ($(TURBINE_ENABLED),false)
$(eval $(call copy-one-file,$(LOCAL_FULL_CLASSES_JACOCO_JAR),$(full_classes_header_jar)))
endif
#######################################
# Capture deps added after base_rules.mk
include $(BUILD_NOTICE_FILE)
#######################################

5
core/soong_app_prebuilt.mk
View file

@ -258,8 +258,3 @@ ifdef LOCAL_PREBUILT_COVERAGE_ARCHIVE
endif
SOONG_ALREADY_CONV += $(LOCAL_MODULE)
#######################################
# Capture deps added after base_rules.mk
include $(BUILD_NOTICE_FILE)
#######################################

5
core/soong_java_prebuilt.mk
View file

@ -209,8 +209,3 @@ $(my_exported_sdk_libs_file):
$(hide) touch $@)
SOONG_ALREADY_CONV += $(LOCAL_MODULE)
#######################################
# Capture deps added after base_rules.mk
include $(BUILD_NOTICE_FILE)
#######################################