mapi: Fix Android build
authorChad Versace <chad.versace@linux.intel.com>
Tue, 10 Apr 2012 22:36:07 +0000 (15:36 -0700)
committerChad Versace <chad.versace@linux.intel.com>
Wed, 11 Apr 2012 04:07:12 +0000 (21:07 -0700)
The Android build was broken by
    commit ca760181b4420696c7e86aa2951d7203522ad1e8
    Author: Kristian Høgsberg <krh@bitplanet.net>
    Date:   Fri Mar 16 12:55:40 2012 -0400

    shared-glapi: Convert to automake

The offending change was that it redefined the filepaths in sources.mak
like this:
    - FOO_FILES := bar.c
    + FOO_FILES := $(TOP)/src/mapi/mapi/bar.c
This broke the build because source filepaths in Android makefiles must be
relative to the makefile.

Ideally, this could be fixed by reverting the change in sources.mak and
making shared-glapi's Makefile.am use $(addprefix $(TOP)/src/mapi/mapi,
$(FOO_FILES)). However, automake doesn't understand builtin GNU make
functions, such as addprefix. So, it seems that automake and Android can
no longer share sources.mak.

Fix the build by duplicating the source lists from sources.mak into
Android.mk.

Signed-off-by: Chad Versace <chad.versace@linux.intel.com>
src/mapi/Android.mk

index b75361f46fad19f2ca757332e5c1da16ee943483..d1749a2625d661cd6b91e1a1b1d51d350b78497a 100644 (file)
@@ -25,9 +25,6 @@
 
 LOCAL_PATH := $(call my-dir)
 
-# get MAPI_GLAPI_FILES
-include $(LOCAL_PATH)/mapi/sources.mak
-
 mapi_abi_headers :=
 
 # ---------------------------------------
@@ -38,7 +35,13 @@ include $(CLEAR_VARS)
 
 abi_header := shared-glapi/glapi_mapi_tmp.h
 
-LOCAL_SRC_FILES := $(MAPI_GLAPI_FILES)
+LOCAL_SRC_FILES := \
+       mapi/entry.c \
+       mapi/mapi_glapi.c \
+       mapi/stub.c \
+       mapi/table.c \
+       mapi/u_current.c \
+       mapi/u_execmem.c
 
 LOCAL_CFLAGS := \
        -DMAPI_MODE_GLAPI \