From 09bcb0ab4b79273cfccdfa87f860b0be1addcc7b Mon Sep 17 00:00:00 2001
From: Rafael Barbalho <rafael.barbalho@intel.com>
Date: Fri, 31 Jan 2014 14:57:40 +0000
Subject: [PATCH] android: Change tests & tools directory to use the lib
 directory

Instead of recompiling the lib directory for every tool or tests we can just
re-use the static library. This also has the nice side effect of fixing the
android "mm" command to allow android users to only rebuild the local changes
in either the test or tools directory.

Signed-off-by: Rafael Barbalho <rafael.barbalho@intel.com>
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
---
 Android.mk       |  5 +----
 lib/Android.mk   |  2 +-
 tests/Android.mk | 51 +++++-------------------------------------------
 tools/Android.mk | 19 +++++-------------
 4 files changed, 12 insertions(+), 65 deletions(-)

diff --git a/Android.mk b/Android.mk
index 7b42d10d..8aeb2d4d 100644
--- a/Android.mk
+++ b/Android.mk
@@ -1,5 +1,2 @@
-LOCAL_PATH := $(call my-dir)
-
-include $(LOCAL_PATH)/tests/Android.mk
-include $(LOCAL_PATH)/tools/Android.mk
+include $(call all-named-subdir-makefiles, lib tests tools)
 
diff --git a/lib/Android.mk b/lib/Android.mk
index 724d8841..67352553 100644
--- a/lib/Android.mk
+++ b/lib/Android.mk
@@ -48,7 +48,7 @@ LOCAL_GENERATED_SOURCES :=       \
 LOCAL_C_INCLUDES +=              \
 	$(LOCAL_PATH)/..
 
-LOCAL_EXPORT_C_INCLUDE_DIRS += $(LOCAL_PATH)
+LOCAL_EXPORT_C_INCLUDE_DIRS := $(LOCAL_PATH)
 
 LOCAL_CFLAGS += -DHAVE_LIBDRM_ATOMIC_PRIMITIVES
 LOCAL_CFLAGS += -DHAVE_STRUCT_SYSINFO_TOTALRAM
diff --git a/tests/Android.mk b/tests/Android.mk
index 0e292a9e..30be4a6d 100644
--- a/tests/Android.mk
+++ b/tests/Android.mk
@@ -1,56 +1,13 @@
-include $(LOCAL_PATH)/tests/Makefile.sources
-include $(LOCAL_PATH)/lib/Makefile.sources
+LOCAL_PATH := $(call my-dir)
 
-skip_lib_list := \
-    igt_kms.c \
-    igt_kms.h
-
-lib_list := $(filter-out $(skip_lib_list),$(libintel_tools_la_SOURCES))
-LIB_SOURCES := $(addprefix lib/,$(lib_list))
-GPU_TOOLS_PATH := $(LOCAL_PATH)
-
-.PHONY: version.h.tmp
-
-$(LOCAL_PATH)/version.h.tmp:
-	@touch $@
-	@if test -d .git; then \
-		if which git > /dev/null; then git log -n 1 --oneline | \
-		        sed 's/^\([^ ]*\) .*/#define IGT_GIT_SHA1 "g\1"/' \
-		        >> $@ ; \
-		fi \
-	else \
-		echo '#define IGT_GIT_SHA1 "NOT-GIT"' >> $@ ; \
-	fi
-
-$(LOCAL_PATH)/version.h: $(LOCAL_PATH)/version.h.tmp
-	@echo "updating version.h"
-	@if ! cmp -s $(GPU_TOOLS_PATH)/version.h.tmp $(GPU_TOOLS_PATH)/version.h; then \
-		mv $(GPU_TOOLS_PATH)/version.h.tmp $(GPU_TOOLS_PATH)/version.h ; \
-	else \
-		rm $(GPU_TOOLS_PATH)/version.h.tmp ; \
-	fi
-
-# FIXME: autogenerate this info #
-$(LOCAL_PATH)/config.h:
-	@echo "updating config.h"
-	echo '#define PACKAGE_VERSION "1.5"' >> $@ ; \
-	echo '#define TARGET_CPU_PLATFORM "android-ia"' >> $@ ;
+include $(LOCAL_PATH)/Makefile.sources
 
 #================#
 
 define add_test
     include $(CLEAR_VARS)
 
-    LOCAL_SRC_FILES :=          \
-       tests/$1.c               \
-       $(LIB_SOURCES)
-
-    LOCAL_GENERATED_SOURCES :=       \
-       $(LOCAL_PATH)/version.h       \
-       $(LOCAL_PATH)/config.h
-
-    LOCAL_C_INCLUDES +=              \
-       $(LOCAL_PATH)/lib
+    LOCAL_SRC_FILES := $1.c
 
     LOCAL_CFLAGS += -DHAVE_STRUCT_SYSINFO_TOTALRAM
     LOCAL_CFLAGS += -DANDROID -UNDEBUG -include "check-ndebug.h"
@@ -63,6 +20,8 @@ define add_test
     LOCAL_MODULE := $1
     LOCAL_MODULE_TAGS := optional
 
+    LOCAL_STATIC_LIBRARIES := libintel_gpu_tools
+
     LOCAL_SHARED_LIBRARIES := libpciaccess  \
                               libdrm        \
                               libdrm_intel
diff --git a/tools/Android.mk b/tools/Android.mk
index 7227c896..a957ec13 100644
--- a/tools/Android.mk
+++ b/tools/Android.mk
@@ -1,24 +1,13 @@
-include $(LOCAL_PATH)/tools/Makefile.sources
-include $(LOCAL_PATH)/lib/Makefile.sources
+LOCAL_PATH := $(call my-dir)
 
-skip_lib_list := \
-    igt_kms.c \
-    igt_kms.h
-
-lib_list := $(filter-out $(skip_lib_list),$(libintel_tools_la_SOURCES))
-LIB_SOURCES := $(addprefix lib/,$(lib_list))
+include $(LOCAL_PATH)/Makefile.sources
 
 #================#
 
 define add_tool
     include $(CLEAR_VARS)
 
-    LOCAL_SRC_FILES :=          \
-       tools/$1.c               \
-       $(LIB_SOURCES)
-
-    LOCAL_C_INCLUDES +=              \
-       $(LOCAL_PATH)/lib
+    LOCAL_SRC_FILES := $1.c
 
     LOCAL_CFLAGS += -DHAVE_TERMIOS_H
     LOCAL_CFLAGS += -DHAVE_STRUCT_SYSINFO_TOTALRAM
@@ -32,6 +21,8 @@ define add_tool
     LOCAL_MODULE := $1
     LOCAL_MODULE_TAGS := optional
 
+    LOCAL_STATIC_LIBRARIES := libintel_gpu_tools
+
     LOCAL_SHARED_LIBRARIES := libpciaccess  \
                               libdrm        \
                               libdrm_intel