[Intel-gfx] [PATCH i-g-t 01/12] meson: Install tests into $libexecdir

Jordan Justen jordan.l.justen at intel.com
Thu Sep 14 18:04:14 UTC 2017


On 2017-09-11 10:24:56, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> autotools installs the tests into $libexecdir. Make meson do the same.
> 
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> ---
>  tests/meson.build | 10 ++++++++--
>  1 file changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/meson.build b/tests/meson.build
> index 4dd5a9c9d4c7..1f98f2a02bb9 100644
> --- a/tests/meson.build
> +++ b/tests/meson.build
> @@ -255,13 +255,19 @@ if alsa.found() and gsl.found()
>         test_deps += alsa
>  endif
>  
> +libexecdir = join_paths(get_option('prefix'), get_option('libexecdir'), 'intel-gpu-tools')
> +
>  foreach prog : test_progs
>         executable(prog, prog + '.c',
> -                       dependencies : test_deps)
> +                  dependencies : test_deps,
> +                  install_dir : libexecdir,
> +                  install : true)
>  endforeach
>  
>  executable('testdisplay', ['testdisplay.c', 'testdisplay_hotplug.c'],
> -               dependencies : test_deps)
> +           dependencies : test_deps,
> +          install_dir : libexecdir,
> +          install : true)

I noticed indentation alignment issues, like this one, in many of the
patches. (Often when adding 'install : true')

Can you review and fix the alignment on all patches?

Series Reviewed-by: Jordan Justen <jordan.l.justen at intel.com>

>  test_progs += 'testdisplay'
>  
>  run_command('generate_testlist.sh', test_progs)
> -- 
> 2.13.5
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx


More information about the Intel-gfx mailing list