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

Commit 8f457381 authored by Cole Faust's avatar Cole Faust
Browse files

Deprecate TARGET_BUILD_APPS=all 

We'd like to convert more of the packaging logic to soong as we work
to get rid of make. TARGET_BUILD_APPS=all makes that tricky, because
we wouldn't know if there was a make-defined app that needed to be
included in the apps package. It doesn't appear to be used, so
deprecate it.

Bug: 376727180
Test: Presubmits
Change-Id: Ie701cbd847260959a537938e2c29c696d107b9b0
parent bfe94d67

core/main.mk

+3 −2
Original line number Diff line number Diff line
@@ -1195,8 +1195,9 @@ endif 
ifneq ($(TARGET_BUILD_APPS),)

  # If this build is just for apps, only build apps and not the full system by default.

  ifneq ($(filter all,$(TARGET_BUILD_APPS)),)

    # If they used the magic goal "all" then build all apps in the source tree.

    unbundled_build_modules := $(foreach m,$(sort $(ALL_MODULES)),$(if $(filter APPS,$(ALL_MODULES.$(m).CLASS)),$(m)))

    # The magic goal "all" used to build all apps in the source tree. This was deprecated

    # so that we can know all TARGET_BUILD_APPS apps are built with soong for soong-only builds.

    $(error TARGET_BUILD_APPS=all is deprecated)

  else

    unbundled_build_modules := $(sort $(TARGET_BUILD_APPS))

  endif