[Intel-gfx] Fwd: Re: [RFC i-g-t 6/7] demos/Makefile: Don't build tools that depend on libdrm_intel.

[Robert Foss]

On Wednesday, May 25, 2016 19:18 BST, robert.foss at collabora.com wrote:

> From: Robert Foss <robert.foss at collabora.com>
>
> Use the HAS_INTEL automake flag to avoid building tools that won't

> compile unless libdrm_intel is available in the build system.
>
> Signed-off-by: Robert Foss <robert.foss at collabora.com>
> ---
>  demos/Android.mk       | 5 ++++-
>  demos/Makefile.am      | 8 +++++---
>  demos/Makefile.sources | 7 +++++++
>  3 files changed, 16 insertions(+), 4 deletions(-)
>  create mode 100644 demos/Makefile.sources
>
> diff --git a/demos/Android.mk b/demos/Android.mk
> index 7d06c9a..add2414 100644
> --- a/demos/Android.mk
> +++ b/demos/Android.mk
> @@ -4,8 +4,11 @@ LOCAL_PATH := $(call my-dir)
>
>  include $(CLEAR_VARS)
>
> -LOCAL_SRC_FILES := intel_sprite_on.c
> +LOCAL_SRC_FILES :=
>
> +ifeq ($(HAVE_LIBDRM_INTEL),true)
> +    LOCAL_SRC_FILES += $(LIBDRM_INTEL_BIN)
> +endif
>
Note for the future, you don't have to do anything.

As/if one tries to get IGT on Android without libdrm_intel and there 
aren't any non-intel demos, the guard should cover the whole file. I 
doubt their build system will be happy if you request a program with no 
sources and just static library.

-Emil