From: Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Date: Thu, 15 Oct 2020 13:53:57 +0000 (+0200)
Subject: meson: enable oomd by default in developer mode
X-Git-Tag: v247-rc1~63^2~1
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=d58c5f0fe7159a05a46c6c791d3350c651600b5d;p=thirdparty%2Fsystemd.git

meson: enable oomd by default in developer mode

We want to compile the new code in CI without having to explicitly specify
-Doomd=true everywhere. Let's enable it by default, and rely on distros
setting -Dmode=release to not have it enabled by default.
---

diff --git a/meson.build b/meson.build
index 43cf7bf2bbf..7940a70c001 100644
--- a/meson.build
+++ b/meson.build
@@ -1412,7 +1412,15 @@ conf.set10('ENABLE_HOMED', have)
 have = have and conf.get('HAVE_PAM') == 1
 conf.set10('ENABLE_PAM_HOME', have)
 
-have = get_option('oomd') and get_option('mode') == 'developer'
+have = get_option('oomd')
+if have == 'auto'
+        have = get_option('mode') == 'developer'
+else
+        have = have == 'true'
+        if have and get_option('mode') != 'developer'
+                error('oomd is not available in release mode (yet)')
+        endif
+endif
 conf.set10('ENABLE_OOMD', have)
 
 want_remote = get_option('remote')
diff --git a/meson_options.txt b/meson_options.txt
index a6a0c1e4b88..59248c70999 100644
--- a/meson_options.txt
+++ b/meson_options.txt
@@ -97,7 +97,7 @@ option('coredump', type : 'boolean',
        description : 'install the coredump handler')
 option('pstore', type : 'boolean',
        description : 'install the pstore archival tool')
-option('oomd', type : 'boolean', value : 'false',
+option('oomd', type : 'combo', choices : ['auto', 'true', 'false'],
        description : 'install the userspace oom killer')
 option('logind', type : 'boolean',
        description : 'install the systemd-logind stack')