diff options
Diffstat (limited to 'core/tasks/platform_availability_check.mk')
| -rw-r--r-- | core/tasks/platform_availability_check.mk | 31 |
1 files changed, 28 insertions, 3 deletions
diff --git a/core/tasks/platform_availability_check.mk b/core/tasks/platform_availability_check.mk index 043d130440..1524758944 100644 --- a/core/tasks/platform_availability_check.mk +++ b/core/tasks/platform_availability_check.mk @@ -17,6 +17,9 @@ # Check whether there is any module that isn't available for platform # is installed to the platform. +# Skip for unbundled builds that don't produce a platform image. +ifeq (,$(TARGET_BUILD_UNBUNDLED)) + # Filter FAKE and NON_INSTALLABLE modules out and then collect those are not # available for platform _modules_not_available_for_platform := \ @@ -26,11 +29,33 @@ $(strip $(foreach m,$(product_MODULES),\ $(if $(filter true,$(ALL_MODULES.$(m).NOT_AVAILABLE_FOR_PLATFORM)),\ $(m)))))) -_violators_with_path := $(foreach m,$(sort $(_modules_not_available_for_platform)),\ +ifndef ALLOW_MISSING_DEPENDENCIES + _violators_with_path := $(foreach m,$(sort $(_modules_not_available_for_platform)),\ $(m):$(word 1,$(ALL_MODULES.$(m).PATH))\ -) + ) -$(call maybe-print-list-and-error,$(_violators_with_path),\ + $(call maybe-print-list-and-error,$(_violators_with_path),\ Following modules are requested to be installed. But are not available \ for platform because they do not have "//apex_available:platform" or \ they depend on other modules that are not available for platform) + +else + +# Don't error out immediately when ALLOW_MISSING_DEPENDENCIES is set. +# Instead, add a dependency on a rule that prints the error message. + define not_available_for_platform_rule + not_installable_file := $(patsubst $(OUT_DIR)/%,$(OUT_DIR)/NOT_AVAILABLE_FOR_PLATFORM/%,$(1)) + $(1): $$(not_installable_file) + $$(not_installable_file): + $(call echo-error,$(2),Module is requested to be installed but is not \ +available for platform because it does not have "//apex_available:platform" or \ +it depends on other modules that are not available for platform.) + exit 1 + endef + + $(foreach m,$(_modules_not_available_for_platform),\ + $(foreach i,$(filter-out $(HOST_OUT)/%,$(ALL_MODULES.$(m).INSTALLED)),\ + $(eval $(call not_available_for_platform_rule,$(i),$(m))))) +endif + +endif |