Issue 5800 - Add option to specify a list of V8 libraries

libadblockplus-android/jni/Android.mk

 LOCAL_PATH := $(call my-dir)
 
+# Report configuration
+ifeq ($(SHARED_V8_LIB_FILENAMES),)
+# static
+$(info [Configuration] Linking statically with built-in v8)
+else
+# dynamic
+
+comma = ,

[sergei, 2017/10/02 09:12:02]

should it be called something like shared_v8_lib_separator?

[anton, 2017/10/02 09:22:04]

Does not matter for me. Should i rename?

+SHARED_V8_LIB_FILENAMES_LIST = $(subst $(comma), ,$(SHARED_V8_LIB_FILENAMES))

[sergei, 2017/10/02 10:08:01]

Sorry, didn't pay attention to it before. What was the problem with spaces in
comparison with comma, I just think that spaces are not only a common way but
also can simplify the method by removing calls of subst.

[sergei, 2017/10/02 12:06:52]

It's three times here and there is even no need in SHARED_V8_LIB_FILENAMES_LIST.
I think it's better to wait for a third opinion.

[diegocarloslima, 2017/10/10 13:40:51]

Yeah, I agree that for sake of simplicity, just a space would be better.

+define info_define
+    $(info [Configuration] Linking dynamically with shared v8 library ./libadblockplus-binaries/android_$(TARGET_ARCH_ABI)/$1)
+endef
+$(foreach item,$(SHARED_V8_LIB_FILENAMES_LIST),$(eval $(call info_define,$(item))))
+endif
+
+# libadblockplus.a
 include $(CLEAR_VARS)
 
 LOCAL_MODULE := libadblockplus
 LOCAL_SRC_FILES := ./libadblockplus-binaries/android_$(TARGET_ARCH_ABI)/libadblockplus.a
 
 include $(PREBUILT_STATIC_LIBRARY)
 
+# libv8-platform.a
+include $(CLEAR_VARS)
+
+LOCAL_MODULE := v8-libplatform
+LOCAL_SRC_FILES := ./libadblockplus-binaries/android_$(TARGET_ARCH_ABI)/libv8_libplatform.a
+
+include $(PREBUILT_STATIC_LIBRARY)
+
+ifeq ($(SHARED_V8_LIB_FILENAMES),)
+# static
+
+# libv8-libsampler.a
 include $(CLEAR_VARS)
 
 LOCAL_MODULE := v8-libsampler
 LOCAL_SRC_FILES := ./libadblockplus-binaries/android_$(TARGET_ARCH_ABI)/libv8_libsampler.a
 
 include $(PREBUILT_STATIC_LIBRARY)
 
+# libv8-base.a
 include $(CLEAR_VARS)
 
 LOCAL_MODULE := v8-base
 LOCAL_SRC_FILES := ./libadblockplus-binaries/android_$(TARGET_ARCH_ABI)/libv8_base.a
 
 include $(PREBUILT_STATIC_LIBRARY)
 
+# libv8_libbase.a
 include $(CLEAR_VARS)
 
 LOCAL_MODULE := v8-libbase
 LOCAL_SRC_FILES := ./libadblockplus-binaries/android_$(TARGET_ARCH_ABI)/libv8_libbase.a
 
 include $(PREBUILT_STATIC_LIBRARY)
 
-include $(CLEAR_VARS)
-
-LOCAL_MODULE := v8-libplatform
-LOCAL_SRC_FILES := ./libadblockplus-binaries/android_$(TARGET_ARCH_ABI)/libv8_libplatform.a
-
-include $(PREBUILT_STATIC_LIBRARY)
-
+# libv8_snapshot.a
 include $(CLEAR_VARS)
 
 LOCAL_MODULE := v8-snapshot
 LOCAL_SRC_FILES := ./libadblockplus-binaries/android_$(TARGET_ARCH_ABI)/libv8_snapshot.a
 
 include $(PREBUILT_STATIC_LIBRARY)
 
+else
+# dynamic
+
+# prebuilt shared libraries v8
+
+SHARED_V8_LIB_FILENAMES_LIST = $(subst $(comma), ,$(SHARED_V8_LIB_FILENAMES))
+define libv8_define
+    include $(CLEAR_VARS)
+
+    LOCAL_MODULE := $1
+    LOCAL_SRC_FILES := ./libadblockplus-binaries/android_$(TARGET_ARCH_ABI)/$1
+
+    include $(PREBUILT_SHARED_LIBRARY)
+endef
+$(foreach item,$(SHARED_V8_LIB_FILENAMES_LIST),$(eval $(call libv8_define,$(item))))
+
+endif
+
 include $(CLEAR_VARS)
 
 LOCAL_MODULE := libadblockplus-jni
 LOCAL_SRC_FILES := JniLibrary.cpp
 LOCAL_SRC_FILES += JniPlatform.cpp
 LOCAL_SRC_FILES += JniJsEngine.cpp JniFilterEngine.cpp JniJsValue.cpp
 LOCAL_SRC_FILES += JniFilter.cpp JniSubscription.cpp JniEventCallback.cpp
 LOCAL_SRC_FILES += JniLogSystem.cpp JniWebRequest.cpp
 LOCAL_SRC_FILES += JniUpdateAvailableCallback.cpp JniUpdateCheckDoneCallback.cpp
 LOCAL_SRC_FILES += JniFilterChangeCallback.cpp JniCallbacks.cpp Utils.cpp
 LOCAL_SRC_FILES += JniNotification.cpp JniShowNotificationCallback.cpp
 LOCAL_SRC_FILES += JniIsAllowedConnectionTypeCallback.cpp
 
-LOCAL_CPP_FEATURES := exceptions 
+LOCAL_CPP_FEATURES := exceptions
 LOCAL_CPPFLAGS += -std=c++11
 
 LOCAL_C_INCLUDES := jni/libadblockplus-binaries/include/
-LOCAL_STATIC_LIBRARIES := libadblockplus v8-base v8-snapshot v8-libsampler v8-libplatform v8-libbase
+
+LOCAL_STATIC_LIBRARIES := libadblockplus v8-libplatform
+
+ifeq ($(SHARED_V8_LIB_FILENAMES),)
+# static
+LOCAL_STATIC_LIBRARIES += v8-base v8-snapshot v8-libsampler v8-libbase
+else
+# dynamic
+SHARED_V8_LIB_FILENAMES_LIST = $(subst $(comma), ,$(SHARED_V8_LIB_FILENAMES))
+LOCAL_STATIC_LIBRARIES += $(SHARED_V8_LIB_FILENAMES_LIST)
+endif
 
 include $(BUILD_SHARED_LIBRARY)