diff options
| author | Ying Wang <wangying@google.com> | 2014-02-12 01:31:21 +0000 |
|---|---|---|
| committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2014-02-12 01:31:22 +0000 |
| commit | bec954d05e0eeec178c796d2332ee9a86becb5dc (patch) | |
| tree | 531d0bf194d281388806ff08ffae5e86cac03a85 /core/main.mk | |
| parent | b12cce10fab8c9adda213506be4ce675104655da (diff) | |
| parent | d18e570fbaef9fdc6605ed75960641416a370a79 (diff) | |
| download | build-bec954d05e0eeec178c796d2332ee9a86becb5dc.zip build-bec954d05e0eeec178c796d2332ee9a86becb5dc.tar.gz build-bec954d05e0eeec178c796d2332ee9a86becb5dc.tar.bz2 | |
Merge "Clean up dangling module names"
Diffstat (limited to 'core/main.mk')
| -rw-r--r-- | core/main.mk | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/core/main.mk b/core/main.mk index df2dea8..871f51b 100644 --- a/core/main.mk +++ b/core/main.mk @@ -753,9 +753,13 @@ ifdef is_sdk_build # Ensure every module listed in PRODUCT_PACKAGES* gets something installed # TODO: Should we do this for all builds and not just the sdk? + dangling_modules := $(foreach m, $(PRODUCTS.$(INTERNAL_PRODUCT).PRODUCT_PACKAGES), \ $(if $(strip $(ALL_MODULES.$(m).INSTALLED)),,\ - $(error $(ALL_MODULES.$(m).MAKEFILE): Module '$(m)' in PRODUCT_PACKAGES has nothing to install!))) + $(eval dangling_modules += $(m)))) + ifneq ($(dangling_modules),) + $(error Module names '$(dangling_modules)' in PRODUCT_PACKAGES has nothing to install!) + endif $(foreach m, $(PRODUCTS.$(INTERNAL_PRODUCT).PRODUCT_PACKAGES_DEBUG), \ $(if $(strip $(ALL_MODULES.$(m).INSTALLED)),,\ $(warning $(ALL_MODULES.$(m).MAKEFILE): Module '$(m)' in PRODUCT_PACKAGES_DEBUG has nothing to install!))) |