[Intel-gfx] [RFC i-g-t 2/9] benchmarks/Makefile: Don't build benchmarks that depend on libdrm_intel.

[Daniel Vetter]

On Mon, May 23, 2016 at 03:04:16PM +0100, Emil Velikov wrote:
> On Friday, May 20, 2016 23:59 BST, robert.foss at collabora.com wrote:
> > From: Robert Foss <robert.foss at collabora.com>
> >
> > Use the HAS_INTEL automake flag to avoid building benchmarks that won't
> > compile unless libdrm_intel is available in the build system.
> >
> > Signed-off-by: Robert Foss <robert.foss at collabora.com>
> > ---
> >  benchmarks/Makefile.sources | 15 ++++++++++-----
> >  1 file changed, 10 insertions(+), 5 deletions(-)
> >
> > diff --git a/benchmarks/Makefile.sources b/benchmarks/Makefile.sources
> > index 81607a5..26ee3ea 100644
> > --- a/benchmarks/Makefile.sources
> > +++ b/benchmarks/Makefile.sources
> > @@ -1,10 +1,6 @@
> >  benchmarksdir=$(libexecdir)/intel-gpu-tools/benchmarks
> >
> >  benchmarks_PROGRAMS =			\
> > -	intel_upload_blit_large         \
> > -	intel_upload_blit_large_gtt     \
> > -	intel_upload_blit_large_map     \
> > -	intel_upload_blit_small		\
> >  	gem_blt				\
> >  	gem_create			\
> >  	gem_exec_ctx			\
> > @@ -16,6 +12,15 @@ benchmarks_PROGRAMS =			\
> >  	gem_prw				\
> >  	gem_set_domain			\
> >  	gem_syslatency			\
> > -	gem_userptr_benchmark		\
> >  	kms_vblank			\
> >  	$(NULL)
> > +
> > +if HAVE_INTEL
> > +	benchmarks_PROGRAMS +=			\
> > +		intel_upload_blit_large		\
> > +		intel_upload_blit_large_gtt	\
> > +		intel_upload_blit_large_map	\
> > +		intel_upload_blit_small		\
> > +		gem_userptr_benchmark		\
> > +		$(NULL)
> > +endif
> Some suggestions:
>  - Please don't use conditionals in the Makefile.sources - you'll break the other build (Android)
>  - instead of ^^ use separate variable and combine them in the Makefile.am/Android.mk
>  - Don't double-indent, don't think any other place does so.

Atm the other build is Intel-only, so I don't think it's that bad really -
if we add a #define HAVE_INTEL there before including Makefile.sources (or
whatever the make magic for this is). If someone wants to build igt on
other Android machines that needs some extensions, but hey not my problem
when Android wants to reinvent half of autoconf ;-)
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch