[Intel-gfx] [PATCH i-g-t] tools/Makefile.sources: fix igt_stats complie error on android

Morton, Derek J derek.j.morton at intel.com
Fri Jul 24 08:55:21 PDT 2015


Found a simpler way of doing this using 'LOCAL_MODULE'. Will prepare a replacement patch.

//Derek

>
>
>-----Original Message-----
>From: Morton, Derek J 
>Sent: Thursday, July 23, 2015 1:59 PM
>To: intel-gfx at lists.freedesktop.org
>Cc: Wood, Thomas; Gore, Tim; Morton, Derek J
>Subject: [PATCH i-g-t] tools/Makefile.sources: fix igt_stats complie error on android
>
>There are two versions of igt_stats.c in tools and lib\tests which causes the build system to have two build modules with the same name.
>This patch specifies a different target name for the tool so there is no clash.
>
>Signed-off-by: Derek Morton <derek.j.morton at intel.com>
>---
> tools/Makefile.sources | 8 +++++++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
>
>diff --git a/tools/Makefile.sources b/tools/Makefile.sources index 8ca9351..d19ef96 100644
>--- a/tools/Makefile.sources
>+++ b/tools/Makefile.sources
>@@ -5,7 +5,7 @@ noinst_PROGRAMS = \
> 	$(NULL)
> 
> bin_PROGRAMS = 				\
>-	igt_stats			\
>+	igt_statistics			\
> 	intel_audio_dump 		\
> 	intel_reg			\
> 	intel_backlight 		\
>@@ -63,3 +63,9 @@ intel_l3_parity_SOURCES =	\
> 	intel_l3_parity.h	\
> 	intel_l3_udev_listener.c
> 
>+# Android does not like building multiple targets with the same name # 
>+tools/igt_stats.c clashes with lib/tests/igt_stats.c. Simplest fix is 
>+to # specify a different target name here.
>+igt_statistics_SOURCES =	\
>+	igt_stats.c
>+
>--
>1.9.1
>
>


More information about the Intel-gfx mailing list