Fix makefiles for Andoid K

It's not safe to rely on LOCAL_MODULE_RELATIVE_PATH on all branches Cleaned up some unnecessary ifdef's Bug b/75229322 Test: Local build of K Fixes build break ab/4660124 Change-Id: I48d3400a2109cd5a2cf593e03f9e234f490c8252 Reviewed-on: https://swiftshader-review.googlesource.com/18189Tested-by: Nicolas Capens <nicolascapens@google.com> Reviewed-by: Nicolas Capens <nicolascapens@google.com>

Fix makefiles for Andoid K
f7a3ff04 · Greg Hartman · Nicolas Capens · e3490ede · f7a3ff04 · f7a3ff04
Commit f7a3ff04 authored Mar 16, 2018 by Greg Hartman Committed by Nicolas Capens Apr 10, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 42 additions and 12 deletions

Android.mk src/OpenGL/libEGL/Android.mk +14 -4

Android.mk src/OpenGL/libGLES_CM/Android.mk +14 -4

Android.mk src/OpenGL/libGLESv2/Android.mk +14 -4

No files found.
--- a/src/OpenGL/libEGL/Android.mk
+++ b/src/OpenGL/libEGL/Android.mk
@@ -69,12 +69,17 @@ COMMON_LDFLAGS := \

 include $(CLEAR_VARS)
 LOCAL_MODULE := libEGL_swiftshader_debug
-ifdef TARGET_2ND_ARCH
+
 ifeq ($(TARGET_TRANSLATE_2ND_ARCH),true)
 LOCAL_MULTILIB := first
 endif
-endif
+
+ifeq (HasRelativePath,$(shell test $(PLATFORM_SDK_VERSION) -ge 21 && echo HasRelativePath))
 LOCAL_MODULE_RELATIVE_PATH := egl
+else
+LOCAL_MODULE_PATH := $(TARGET_OUT_SHARED_LIBRARIES)/egl
+endif
+
 LOCAL_VENDOR_MODULE := true
 LOCAL_MODULE_TAGS := optional
 LOCAL_CLANG := true
@@ -89,12 +94,17 @@ include $(BUILD_SHARED_LIBRARY)

 include $(CLEAR_VARS)
 LOCAL_MODULE := libEGL_swiftshader
-ifdef TARGET_2ND_ARCH
+
 ifeq ($(TARGET_TRANSLATE_2ND_ARCH),true)
 LOCAL_MULTILIB := first
 endif
-endif
+
+ifeq (HasRelativePath,$(shell test $(PLATFORM_SDK_VERSION) -ge 21 && echo HasRelativePath))
 LOCAL_MODULE_RELATIVE_PATH := egl
+else
+LOCAL_MODULE_PATH := $(TARGET_OUT_SHARED_LIBRARIES)/egl
+endif
+
 LOCAL_VENDOR_MODULE := true
 LOCAL_MODULE_TAGS := optional
 LOCAL_CLANG := true

--- a/src/OpenGL/libGLES_CM/Android.mk
+++ b/src/OpenGL/libGLES_CM/Android.mk
@@ -92,12 +92,17 @@ COMMON_LDFLAGS := \

 include $(CLEAR_VARS)
 LOCAL_MODULE := libGLESv1_CM_swiftshader_debug
-ifdef TARGET_2ND_ARCH
+
 ifeq ($(TARGET_TRANSLATE_2ND_ARCH),true)
 LOCAL_MULTILIB := first
 endif
-endif
+
+ifeq (HasRelativePath,$(shell test $(PLATFORM_SDK_VERSION) -ge 21 && echo HasRelativePath))
 LOCAL_MODULE_RELATIVE_PATH := egl
+else
+LOCAL_MODULE_PATH := $(TARGET_OUT_SHARED_LIBRARIES)/egl
+endif
+
 LOCAL_VENDOR_MODULE := true
 LOCAL_MODULE_TAGS := optional
 LOCAL_CLANG := true
@@ -112,12 +117,17 @@ include $(BUILD_SHARED_LIBRARY)

 include $(CLEAR_VARS)
 LOCAL_MODULE := libGLESv1_CM_swiftshader
-ifdef TARGET_2ND_ARCH
+
 ifeq ($(TARGET_TRANSLATE_2ND_ARCH),true)
 LOCAL_MULTILIB := first
 endif
-endif
+
+ifeq (HasRelativePath,$(shell test $(PLATFORM_SDK_VERSION) -ge 21 && echo HasRelativePath))
 LOCAL_MODULE_RELATIVE_PATH := egl
+else
+LOCAL_MODULE_PATH := $(TARGET_OUT_SHARED_LIBRARIES)/egl
+endif
+
 LOCAL_VENDOR_MODULE := true
 LOCAL_MODULE_TAGS := optional
 LOCAL_CLANG := true

--- a/src/OpenGL/libGLESv2/Android.mk
+++ b/src/OpenGL/libGLESv2/Android.mk
@@ -101,12 +101,17 @@ COMMON_LDFLAGS := \

 include $(CLEAR_VARS)
 LOCAL_MODULE := libGLESv2_swiftshader_debug
-ifdef TARGET_2ND_ARCH
+
 ifeq ($(TARGET_TRANSLATE_2ND_ARCH),true)
 LOCAL_MULTILIB := first
 endif
-endif
+
+ifeq (HasRelativePath,$(shell test $(PLATFORM_SDK_VERSION) -ge 21 && echo HasRelativePath))
 LOCAL_MODULE_RELATIVE_PATH := egl
+else
+LOCAL_MODULE_PATH := $(TARGET_OUT_SHARED_LIBRARIES)/egl
+endif
+
 LOCAL_VENDOR_MODULE := true
 LOCAL_MODULE_TAGS := optional
 LOCAL_CLANG := true
@@ -121,12 +126,17 @@ include $(BUILD_SHARED_LIBRARY)

 include $(CLEAR_VARS)
 LOCAL_MODULE := libGLESv2_swiftshader
-ifdef TARGET_2ND_ARCH
+
 ifeq ($(TARGET_TRANSLATE_2ND_ARCH),true)
 LOCAL_MULTILIB := first
 endif
-endif
+
+ifeq (HasRelativePath,$(shell test $(PLATFORM_SDK_VERSION) -ge 21 && echo HasRelativePath))
 LOCAL_MODULE_RELATIVE_PATH := egl
+else
+LOCAL_MODULE_PATH := $(TARGET_OUT_SHARED_LIBRARIES)/egl
+endif
+
 LOCAL_VENDOR_MODULE := true
 LOCAL_MODULE_TAGS := optional
 LOCAL_CLANG := true