Re: [PATCH libdrm] meson: do not use cairo/valgrind if disabled

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sunday, 2018-02-18 14:00:50 +0100, Igor Gnatenko wrote:
> -Dcairo-tests=false currently results into enabling cairo support if it
> was found. Same for valgrind.

Indeed, this was wrong; thanks for the fix!
Reviewed-by: Eric Engestrom <eric.engestrom@xxxxxxxxxx>

Do you have commit access, or do you want me to push this for you?

> 
> Signed-off-by: Igor Gnatenko <ignatenko@xxxxxxxxxx>
> ---
>  meson.build | 20 ++++++++++++++++----
>  1 file changed, 16 insertions(+), 4 deletions(-)
> 
> diff --git a/meson.build b/meson.build
> index 166559e8..695f89b3 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -226,8 +226,20 @@ endforeach
>  
>  dep_pciaccess = dependency('pciaccess', version : '>= 0.10', required : with_intel)
>  dep_cunit = dependency('cunit', version : '>= 2.1', required : false)
> -dep_cairo = dependency('cairo', required : with_cairo_tests == 'true')
> -dep_valgrind = dependency('valgrind', required : with_valgrind == 'true')
> +if with_cairo_tests != 'false'
> +  dep_cairo = dependency('cairo', required : with_cairo_tests == 'true')
> +  with_cairo_tests = dep_cairo.found()
> +else
> +  dep_cairo = declare_dependency()

Nit: `dep_cairo = []` is enough; I'll change that if I'm the one to push it.

> +  with_cairo_tests = false

We try to avoid changing the type of a var; could you send a follow-up
patch to rename the get_option() var to `_cairo_tests`?

(same obviously applies for the valgrind bits)

> +endif
> +if with_valgrind != 'false'
> +  dep_valgrind = dependency('valgrind', required : with_valgrind == 'true')
> +  with_valgrind = dep_valgrind.found()
> +else
> +  dep_valgrind = declare_dependency()
> +  with_valgrind = false
> +endif
>  
>  with_man_pages = get_option('man-pages')
>  prog_xslt = find_program('xsltproc', required : with_man_pages == 'true')
> @@ -259,8 +271,8 @@ foreach t : [
>               [with_radeon, 'RADEON'],
>               [with_vc4, 'VC4'],
>               [with_vmwgfx, 'VMWGFX'],
> -             [dep_cairo.found(), 'CAIRO'],
> -             [dep_valgrind.found(), 'VALGRIND'],
> +             [with_cairo_tests, 'CAIRO'],
> +             [with_valgrind, 'VALGRIND'],
>              ]
>    config.set10('HAVE_@0@'.format(t[1]), t[0])
>  endforeach
> -- 
> 2.16.2
> 
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux