lib/audit_logging: Require jansson JSON library for building the AD DC

This combination is untested and it is reasonable to require this
broadly available library for the AD DC build.

Doing so keeps the combinational complexity down and ensures we test
what we ship.  (It was failing to compile).

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>

diff --git a/lib/audit_logging/wscript b/lib/audit_logging/wscript
index 7818ed0ecfabd5da92e7eb40b3fd5064cad39394..d40fc6f079f21afdb006e23ccd0df8ca7884ba0f 100644 (file)
--- a/lib/audit_logging/wscript
+++ b/lib/audit_logging/wscript
@@ -21,14 +21,16 @@ def configure(conf):
 
     if not conf.CONFIG_GET('HAVE_JSON_OBJECT'):
         if Options.options.with_json_audit != False:
-            conf.fatal("JSON support not found. "
+            conf.fatal("Jansson JSON support not found. "
                        "Try installing libjansson-dev or jansson-devel. "
                        "Otherwise, use --without-json-audit to build without "
                        "JSON support. "
                        "JSON support is required for the JSON "
-                       "formatted audit log feature")
-        if conf.CONFIG_GET('ENABLE_SELFTEST') and \
-          (not Options.options.without_ad_dc):
-            raise Utils.WafError('jansson JSON library required for '
-                                 '--enable-selftest when building the AD DC')
-        Logs.info("Building without jansson json log support")
+                       "formatted audit log feature and the AD DC")
+        if not Options.options.without_ad_dc:
+            raise Utils.WafError('--without-json-audit requires '
+                                 '--without-ad-dc. '
+                                 'Jansson JSON library is required for '
+                                 'building the AD DC')
+        Logs.info("Building without Jansson JSON log support")
+