From: Zbigniew Jędrzejewski-Szmek Date: Wed, 10 Oct 2018 09:56:45 +0000 (+0200) Subject: meson: c++ is required for the fuzzer builds X-Git-Tag: v240~584^2 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=refs%2Fpull%2F10351%2Fhead;p=thirdparty%2Fsystemd.git meson: c++ is required for the fuzzer builds The configuration would fail with an error about cpp being an unknown language anyway, but it's nicer to fail early and explicitly. https://github.com/systemd/systemd/pull/10339#issuecomment-428279175 --- diff --git a/meson.build b/meson.build index 0ebbfce430e..35b6ea0e347 100644 --- a/meson.build +++ b/meson.build @@ -27,6 +27,13 @@ substs = configuration_data() substs.set('PACKAGE_URL', 'https://www.freedesktop.org/wiki/Software/systemd') substs.set('PACKAGE_VERSION', meson.project_version()) +want_ossfuzz = get_option('oss-fuzz') +want_libfuzzer = get_option('llvm-fuzz') +if want_ossfuzz and want_libfuzzer + error('only one of oss-fuzz and llvm-fuzz can be specified') +endif +fuzzer_build = want_ossfuzz or want_libfuzzer + ##################################################################### # Try to install the git pre-commit hook @@ -274,7 +281,7 @@ want_tests = get_option('tests') slow_tests = want_tests != 'false' and get_option('slow-tests') install_tests = get_option('install-tests') -cxx = find_program('c++', required : false) +cxx = find_program('c++', required : fuzzer_build) if cxx.found() # Used only for tests add_languages('cpp') @@ -283,16 +290,9 @@ else cxx_cmd = '' endif -want_ossfuzz = get_option('oss-fuzz') -want_libfuzzer = get_option('llvm-fuzz') -fuzzer_build = want_ossfuzz or want_libfuzzer -if want_ossfuzz and want_libfuzzer - error('only one of oss-fuzz and llvm-fuzz can be specified') -endif if want_libfuzzer fuzzing_engine = meson.get_compiler('cpp').find_library('Fuzzer') -endif -if want_ossfuzz +elif want_ossfuzz fuzzing_engine = meson.get_compiler('cpp').find_library('FuzzingEngine') endif