diff mbox

[i-g-t] tools/Android.mk: Add zlib support

Message ID 2D016F5BA44C744783BA96B1D3EBC5BD13EFA714@IRSMSX102.ger.corp.intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Derek Morton Jan. 15, 2016, 3:49 p.m. UTC 
Can this be merged so IGT on Android builds? No one has raise any objections since Monday about this patch.

//Derek

-----Original Message-----
From: Morton, Derek J 
Sent: Monday, January 11, 2016 1:32 PM
To: intel-gfx@lists.freedesktop.org
Cc: Wood, Thomas; Gore, Tim; chris@chris-wilson.co.uk; Morton, Derek J
Subject: [PATCH] tools/Android.mk: Add zlib support

IGT does not build for Android due to a zlib dependency being added to intel_error_decode.c in a recent patch.
This patch fixes the error by updating the Android makefile to add the path to the zlib library and using any LDFLAGS specified in Makefile.sources.

Signed-off-by: Derek Morton <derek.j.morton@intel.com>
---
 tools/Android.mk | 5 +++++
 1 file changed, 5 insertions(+)

--
1.9.1

Comments

Lespiau, Damien Jan. 15, 2016, 3:59 p.m. UTC | #1 
On Fri, Jan 15, 2016 at 03:49:00PM +0000, Morton, Derek J wrote:
> Can this be merged so IGT on Android builds? No one has raise any
> objections since Monday about this patch.

Merged, thanks for the patch.
diff mbox

Patch

diff --git a/tools/Android.mk b/tools/Android.mk index 934d3a1..da4f3c0 100644
--- a/tools/Android.mk
+++ b/tools/Android.mk
@@ -23,9 +23,14 @@  define add_tool
     LOCAL_CFLAGS += -Wno-error=return-type
     # Excessive complaining for established cases. Rely on the Linux version warnings.
     LOCAL_CFLAGS += -Wno-sign-compare
+    ifeq ($($(1)_LDFLAGS),)
+    else
+        LOCAL_LDFLAGS += $($(1)_LDFLAGS)
+    endif
 
     LOCAL_C_INCLUDES = $(LOCAL_PATH)/../lib
     LOCAL_C_INCLUDES += ${ANDROID_BUILD_TOP}/external/PRIVATE/drm/include/drm
+    LOCAL_C_INCLUDES += ${ANDROID_BUILD_TOP}/external/zlib
 
     LOCAL_MODULE := $1_tool
     LOCAL_MODULE_TAGS := optional