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

Commit 15e487a7 authored by Ying Wang's avatar Ying Wang
Browse files

Don't build everything for APKCERTS_FILE

Before this change, to build APKCERTS_FILE you have to build any modules
in the system first, which include modules that only need to be checked,
instead of being built. This can reduce the build time when "dist" is
specified.

Change-Id: I64bee67977adcaf7b7c8df3f8f6492c9c33bc69d
parent 7e6d4e45
Loading
Loading
Loading
Loading
+5 −10
Original line number Diff line number Diff line
@@ -215,16 +215,9 @@ name := $(name)-apkcerts-$(FILE_NAME_TAG)
intermediates := \
	$(call intermediates-dir-for,PACKAGING,apkcerts)
APKCERTS_FILE := $(intermediates)/$(name).txt
# Depending on the built packages isn't exactly right,
# but it should guarantee that the apkcerts file is rebuilt
# if any packages change which certs they're signed with.
all_built_packages := $(foreach p,$(PACKAGES),$(ALL_MODULES.$(p).BUILT))
ifneq ($(TARGET_BUILD_APPS),)
# We don't need to really build all the modules for apps_only build.
# We don't need to really build all the modules.
# TODO: rebuild APKCERTS_FILE if any app change its cert.
$(APKCERTS_FILE):
else
$(APKCERTS_FILE): $(all_built_packages)
endif
	@echo APK certs list: $@
	@mkdir -p $(dir $@)
	@rm -f $@
@@ -238,7 +231,9 @@ endif
.PHONY: apkcerts-list
apkcerts-list: $(APKCERTS_FILE)

ifneq (,$(TARGET_BUILD_APPS))
  $(call dist-for-goals, apps_only, $(APKCERTS_FILE):apkcerts.txt)
endif

# -----------------------------------------------------------------
# module info file