meson: do not search for needless deps

If we don't want to use these deps, there's no good reason to search
for them in the first place. This should shave a bit of time for the
initial build.

Signed-off-by: Erik Faye-Lund <kusmabite@gmail.com>
Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com>

diff --git a/meson.build b/meson.build
index 24d997b3e0a761e0d0eae897ae6b3fa418bb8244..d6a2d83b5657879379a9a6f76832523b788650c9 100644 (file)
--- a/meson.build
+++ b/meson.build
@@ -691,9 +691,13 @@ if with_glvnd
 endif
 
 # TODO: make this conditional
-dep_valgrind = dependency('valgrind', required : false)
-if dep_valgrind.found() and with_valgrind
-  pre_args += '-DHAVE_VALGRIND'
+if with_valgrind != 'false'
+  dep_valgrind = dependency('valgrind', required : with_valgrind == 'true')
+  if dep_valgrind.found()
+    pre_args += '-DHAVE_VALGRIND'
+  endif
+else
+  dep_valgrind = []
 endif
 
 # pthread stubs. Lets not and say we didn't
@@ -709,9 +713,13 @@ endif
 
 # TODO: llvm-prefix and llvm-shared-libs
 
-dep_unwind = dependency('libunwind', required : false)
-if dep_unwind.found() and with_libunwind
-  pre_args += '-DHAVE_LIBUNWIND'
+if with_libunwind != 'false'
+  dep_unwind = dependency('libunwind', required : with_libunwind == 'true')
+  if dep_unwind.found()
+    pre_args += '-DHAVE_LIBUNWIND'
+  endif
+else
+  dep_unwind = []
 endif
 
 # TODO: flags for opengl, gles, dri

diff --git a/meson_options.txt b/meson_options.txt
index 74f1e71bf4344351a4c579bf733ce3d034607d59..f6f21b4c02d3542c348c988b8c18124ba1b2893f 100644 (file)
--- a/meson_options.txt
+++ b/meson_options.txt
@@ -138,15 +138,17 @@ option(
 )
 option(
   'valgrind',
-  type : 'boolean',
-  value : true,
-  description : 'Build with valgrind support if possible'
+  type : 'combo',
+  value : 'auto',
+  choices : ['auto', 'true', 'false'],
+  description : 'Build with valgrind support'
 )
 option(
   'libunwind',
-  type : 'boolean',
-  value : true,
-  description : 'Use libunwind for stack-traces if possible'
+  type : 'combo',
+  value : 'auto',
+  choices : ['auto', 'true', 'false'],
+  description : 'Use libunwind for stack-traces'
 )
 option(
   'build-tests',