[PATCH libdrm] meson: do not use cairo/valgrind if disabled
Eric Engestrom
eric.engestrom at imgtec.com
Mon Feb 19 12:15:16 UTC 2018
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 at imgtec.com>
Do you have commit access, or do you want me to push this for you?
>
> Signed-off-by: Igor Gnatenko <ignatenko at redhat.com>
> ---
> 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_ at 0@'.format(t[1]), t[0])
> endforeach
> --
> 2.16.2
>
More information about the dri-devel
mailing list