Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2a982e5b authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Bypass missing required module check on ASAN SANITIZE_TARGET build" am:... 

Merge "Bypass missing required module check on ASAN SANITIZE_TARGET build" am: 15c2fbfc am: 81dddafd

Original change: https://android-review.googlesource.com/c/platform/build/+/1398272

Change-Id: Id5de7d1e96942cc24d2556bfb90b35ac818720b1
parents 4a0f6c33 81dddafd

core/main.mk

+17 −5
Original line number Original line Diff line number Diff line
@@ -699,10 +699,23 @@ $(call select-bitness-of-target-host-required-modules,TARGET,HOST) 
$(call select-bitness-of-target-host-required-modules,HOST,TARGET)

$(call select-bitness-of-target-host-required-modules,HOST,TARGET)

_nonexistent_required := $(sort $(_nonexistent_required))

_nonexistent_required := $(sort $(_nonexistent_required))





check_missing_required_modules := true

ifneq (,$(filter true,$(ALLOW_MISSING_DEPENDENCIES) $(BUILD_BROKEN_MISSING_REQUIRED_MODULES)))

  check_missing_required_modules :=

endif # ALLOW_MISSING_DEPENDENCIES == true || BUILD_BROKEN_MISSING_REQUIRED_MODULES == true



# Some executables are skipped in ASAN SANITIZE_TARGET build, thus breaking their dependencies.

ifneq (,$(filter address,$(SANITIZE_TARGET)))

  check_missing_required_modules :=

endif # SANITIZE_TARGET has ASAN



# HOST OS darwin build is broken, disable this check for darwin for now.

# HOST OS darwin build is broken, disable this check for darwin for now.

# TODO(b/162102724): Remove this

# TODO(b/162102724): Remove this when darwin host has no broken dependency.

ifeq (,$(filter $(HOST_OS),darwin))

ifneq (,$(filter $(HOST_OS),darwin))

ifeq (,$(filter true,$(ALLOW_MISSING_DEPENDENCIES) $(BUILD_BROKEN_MISSING_REQUIRED_MODULES)))

  check_missing_required_modules :=

endif # HOST_OS == darwin



ifeq (true,$(check_missing_required_modules))

ifneq (,$(_nonexistent_required))

ifneq (,$(_nonexistent_required))

  $(warning Missing required dependencies:)

  $(warning Missing required dependencies:)

  $(foreach r_i,$(_nonexistent_required), \

  $(foreach r_i,$(_nonexistent_required), \
@@ -712,8 +725,7 @@ ifneq (,$(_nonexistent_required)) 
  $(warning Set BUILD_BROKEN_MISSING_REQUIRED_MODULES := true to bypass this check if this is intentional)

  $(warning Set BUILD_BROKEN_MISSING_REQUIRED_MODULES := true to bypass this check if this is intentional)

  $(error Build failed)

  $(error Build failed)

endif # _nonexistent_required != empty

endif # _nonexistent_required != empty

endif # ALLOW_MISSING_DEPENDENCIES != true && BUILD_BROKEN_MISSING_REQUIRED_MODULES != true

endif # check_missing_required_modules == true

endif # HOST_OS != darwin





define add-required-deps

define add-required-deps

$(1): | $(2)

$(1): | $(2)