Hi Thomas,
I tried these patches on Android. They worked with some modifications. See comments below.
//Derek
>
>
>-----Original Message-----
>From: Wood, Thomas
>Sent: Thursday, July 16, 2015 12:08 PM
>To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
>Cc: Morton, Derek J
>Subject: [PATCH i-g-t 2/2] tools/Android.mk: add any extra program sources
>
>Cc: Derek Morton <derek.j.morton@xxxxxxxxx>
>Signed-off-by: Thomas Wood <thomas.wood@xxxxxxxxx>
>---
> tools/Android.mk | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/tools/Android.mk b/tools/Android.mk index 39f4512..f7dc3bb 100644
>--- a/tools/Android.mk
>+++ b/tools/Android.mk
>@@ -1,13 +1,13 @@
> LOCAL_PATH := $(call my-dir)
>
>-include $(LOCAL_PATH)/Makefile.sources
>+include ./$(LOCAL_PATH)/Makefile.sources
$(LOCAL_PATH) is an absolute path so putting ./ in front of it breaks things.
>
> #================#
>
> define add_tool
> include $(CLEAR_VARS)
>
>- LOCAL_SRC_FILES := $1.c
>+ LOCAL_SRC_FILES := $1.c $($(1)_SOURCES)
This causes duplication errors as $1.c will also be in $($(1)_SOURCES). However $($(1)_SOURCES) on its own does not work either as it is not defined for all targets.
I ended up with:
ifeq ($($(1)_SOURCES),)
LOCAL_SRC_FILES := $1.c
else
LOCAL_SRC_FILES := $($(1)_SOURCES)
endif
>
> LOCAL_CFLAGS += -DHAVE_TERMIOS_H
> LOCAL_CFLAGS += -DHAVE_STRUCT_SYSINFO_TOTALRAM
>--
>2.4.3
>
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
