[pulseaudio-commits] [Git][pulseaudio/pulseaudio][master] build-sys: Add an option for enabling/disabling Valgrind

[Georg Chini]

Georg Chini pushed to branch master at PulseAudio / pulseaudio


Commits:
dc79f304 by Tanu Kaskinen at 2020-12-09T11:07:09+00:00
build-sys: Add an option for enabling/disabling Valgrind

In OpenEmbedded the PulseAudio recipe currently disables Valgrind
support by passing "ac_cv_header_valgrind_memcheck_h=no" to the
configure script (this was added to make it deterministic whether
Valgrdind support gets enabled or not). I'm converting the PulseAudio
recipe to use Meson, and I needed an option to disable Valgrind.

- - - - -


2 changed files:

- meson.build
- meson_options.txt


Changes:

=====================================
meson.build
=====================================
@@ -221,7 +221,6 @@ check_headers = [
   'sys/un.h',
   'sys/wait.h',
   'syslog.h',
-  'valgrind/memcheck.h',
   'xlocale.h',
 ]
 
@@ -232,6 +231,10 @@ foreach h : check_headers
   endif
 endforeach
 
+if cc.has_header('valgrind/memcheck.h', required: get_option('valgrind'))
+  cdata.set('HAVE_VALGRIND_MEMCHECK_H', 1)
+endif
+
 # FIXME: move this to the above set
 if cc.has_header('pthread.h')
   cdata.set('HAVE_PTHREAD', 1)
@@ -842,6 +845,7 @@ summary = [
   'Enable SoXR (resampler):       @0@'.format(soxr_dep.found()),
   'Enable WebRTC echo canceller:  @0@'.format(webrtc_dep.found()),
   'Enable Gcov coverage:          @0@'.format(get_option('gcov')),
+  'Enable Valgrind:               @0@'.format(cdata.has('HAVE_VALGRIND_MEMCHECK_H')),
   'Enable man pages:              @0@'.format(get_option('man')),
   'Enable unit tests:             @0@'.format(get_option('tests')),
   '',


=====================================
meson_options.txt
=====================================
@@ -141,6 +141,9 @@ option('systemd',
 option('udev',
        type : 'feature', value : 'auto',
        description : 'Optional udev support')
+option('valgrind',
+       type : 'feature', value : 'auto',
+       description : 'Optional Valgrind support')
 option('x11',
        type : 'feature', value : 'auto',
        description : 'Optional X11 support')



View it on GitLab: https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/commit/dc79f304ddc4a2bfcaf7b2d69fa5f1c28dc95ce8

-- 
View it on GitLab: https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/commit/dc79f304ddc4a2bfcaf7b2d69fa5f1c28dc95ce8
You're receiving this email because of your account on gitlab.freedesktop.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/pulseaudio-commits/attachments/20201209/a71a7a54/attachment-0001.htm>