Commit ad460ce9 authored Mar 15, 2017 by Colin Cross

Remove bad make rules causing system.img to rebuild every time

ALL_DEFAULT_INSTALLED_MODULES is supposed to contain paths, not
module names.  Module names are phony targets, which causes
anything that depends on it to be always dirty, resulting in
rebuilding system.img on every incremental build.

ALL_DEFAULT_INSTALLED_MODULES shouldn't be used anyways, use
PRODUCT_PACKAGES in a product makefile instead, and all_modules
isn't used at all any more, remove both of them.

Bug: 36274890
Test: builds
Change-Id: I2db49dcc27515357b0306d0c3e9b38026012b1ea

parent 20421938

cmds/vr/pose/Android.mk

+0 −6

Original line number	Diff line number	Diff line
		@@ -33,9 +33,3 @@ LOCAL_SHARED_LIBRARIES := $(sharedLibraries)
		LOCAL_MODULE := pose
		LOCAL_MODULE_TAGS := optional
		include $(BUILD_EXECUTABLE)

		ifeq ($(TARGET_BUILD_VARIANT),eng)
		ALL_DEFAULT_INSTALLED_MODULES += pose
		all_modules: pose
		endif

cmds/vr/vrscreencap/Android.mk

+0 −5

Original line number	Diff line number	Diff line
		@@ -21,8 +21,3 @@ LOCAL_MODULE := vrscreencap
		LOCAL_MODULE_TAGS := optional

		include $(BUILD_EXECUTABLE)

		ifeq ($(TARGET_BUILD_VARIANT),eng)
		ALL_DEFAULT_INSTALLED_MODULES += vrscreencap
		all_modules: vrscreencap
		endif