meson: Make man page build and install conditional
authorArnaud Rebillout <arnaud.rebillout@collabora.com>
Mon, 5 Nov 2018 05:03:03 +0000 (12:03 +0700)
committerArun Raghavan <arun@arunraghavan.net>
Sat, 8 Jun 2019 09:57:00 +0000 (11:57 +0200)
Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>
man/meson.build
meson.build
meson_options.txt

index 23dd9fe..ef483c7 100644 (file)
@@ -20,17 +20,14 @@ manpages = [
 # FIXME: Add esdcompat if HAVE_ESOUND
 #manpages += ['esdcompat', '1'],
 
-# FIXME: This should be conditional
-want_man = true
-
 man_data = configuration_data()
 man_data.set('PACKAGE_URL', 'http://pulseaudio.org/')
 man_data.set('PACKAGE_BUGREPORT', 'pulseaudio-discuss (at) lists (dot) freedesktop (dot) org')
 man_data.set('PA_DEFAULT_CONFIG_DIR', cdata.get_unquoted('PA_DEFAULT_CONFIG_DIR'))
 
-perl = find_program('perl', required: want_man == true)
+perl = find_program('perl', required: true)
 xmltoman_cmd = [perl, join_paths(meson.current_source_dir(), 'xmltoman')]
-xmllint = find_program('xmllint', required: want_man == true)
+xmllint = find_program('xmllint', required: false)
 
 foreach tuple : manpages
   stem = tuple[0]
@@ -55,7 +52,7 @@ foreach tuple : manpages
     output : man,
     capture : true,
     command : xmltoman_cmd + ['@INPUT@'],
-    install : want_man,
+    install : true,
     install_dir : mandirn,
   )
 
@@ -64,7 +61,7 @@ foreach tuple : manpages
       alias,
       output : alias,
       command : ['ln', '-fs', man, '@OUTPUT@'],
-      install : want_man,
+      install : true,
       install_dir : mandirn,
     )
   endforeach
index f4449b8..2c21cce 100644 (file)
@@ -434,7 +434,9 @@ check_dep = dependency('check', version : '>= 0.9.10', required : get_option('te
 # Now generate config.h from everything above
 configure_file(output : 'config.h', configuration : cdata)
 
-subdir('man')
+if get_option('man')
+  subdir('man')
+endif
 subdir('shell-completion/bash')
 subdir('shell-completion/zsh')
 subdir('src')
@@ -500,6 +502,7 @@ summary = [
   'Enable SoXR (resampler):       @0@'.format(soxr_dep.found()),
   'Enable WebRTC echo canceller:  @0@'.format(webrtc_dep.found()),
 #  'Enable gcov coverage:          @0@'.format(${ENABLE_GCOV}),
+  'Enable man pages:              @0@'.format(get_option('man')),
   'Enable unit tests:             @0@'.format(get_option('tests')),
   '',
   'Database:                      @0@'.format(get_option('database')),
index fadd587..d2a7155 100644 (file)
@@ -1,3 +1,6 @@
+option('man',
+       type : 'boolean',
+       description : 'Enable building and installation of man pages')
 option('tests',
        type : 'boolean',
        description : 'Enable unit tests')