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

Commit 1a6f8016 authored by Dan Willemsen's avatar Dan Willemsen
Browse files

Don't continuously rebuild mediaextractor 

Use LOCAL_REQUIRED_MODULES instead of LOCAL_ADDITIONAL_DEPENDENCIES to
remove the dependency on a phony module. LOCAL_ADDITIONAL_DEPENDENCIES
also rebuilds the objects when the dependency is dirty, instead of just
making sure mediaextractor-seccomp.policy is installed.

Change-Id: I0d21d4e874b55eadaf5dfb395a85c987392250b3
parent a3e696da

services/mediaextractor/Android.mk

+1 −3
Original line number Diff line number Diff line
@@ -11,9 +11,7 @@ include $(BUILD_SHARED_LIBRARY) 


# service executable

include $(CLEAR_VARS)

ifeq ($(TARGET_ARCH), $(filter $(TARGET_ARCH), arm arm64))

LOCAL_ADDITIONAL_DEPENDENCIES += mediaextractor-seccomp.policy

endif

LOCAL_REQUIRED_MODULES_arm := mediaextractor-seccomp.policy

LOCAL_SRC_FILES := main_extractorservice.cpp minijail/minijail.cpp

LOCAL_SHARED_LIBRARIES := libmedia libmediaextractorservice libbinder libutils liblog libicuuc libminijail

LOCAL_STATIC_LIBRARIES := libicuandroid_utils

services/mediaextractor/minijail/Android.mk

+1 −1
Original line number Diff line number Diff line
@@ -4,7 +4,7 @@ ifeq ($(TARGET_ARCH), $(filter $(TARGET_ARCH), arm arm64)) 
include $(CLEAR_VARS)

LOCAL_MODULE := mediaextractor-seccomp.policy

LOCAL_MODULE_CLASS := ETC

LOCAL_MODULE_PATH := $(TARGET_OUT)/etc/seccomp_policy/

LOCAL_MODULE_PATH := $(TARGET_OUT)/etc/seccomp_policy



# mediaextractor runs in 32-bit combatibility mode. For 64 bit architectures,

# use the 32 bit policy