meson: do not search for needless deps
authorErik Faye-Lund <kusmabite@gmail.com>
Wed, 25 Oct 2017 08:02:38 +0000 (10:02 +0200)
committerErik Faye-Lund <kusmabite@gmail.com>
Tue, 31 Oct 2017 14:44:13 +0000 (15:44 +0100)
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>
meson.build
meson_options.txt

index 24d997b..d6a2d83 100644 (file)
@@ -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
index 74f1e71..f6f21b4 100644 (file)
@@ -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',