1
0
mirror of https://github.com/samba-team/samba.git synced 2024-12-22 13:34:15 +03:00

build: Make --with-json-audit the default

Thanks to Rowland for a clear description of the behaviour for the smb.conf manpage.

This means that those not wanting to link to libarchive will just need to
build --without-json-audit.

In general, we prefer that optional libraries be required by default
so that they are not accidentially missed, particularly in packages.

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Gary Lockyer <gary@catalyst.net.nz>
This commit is contained in:
Andrew Bartlett 2018-05-10 13:01:05 +12:00
parent 171750e966
commit 6fda57d309
2 changed files with 10 additions and 6 deletions

View File

@ -4,7 +4,7 @@ import Logs, Options, Utils
import samba3
def set_options(opt):
help = ("Build with JSON auth audit support (default=auto). "
help = ("Build with JSON auth audit support (default=True). "
"This requires the jansson devel package.")
opt.SAMBA3_ADD_OPTION('json-audit', default=None, help=(help))
@ -20,9 +20,13 @@ def configure(conf):
conf.CHECK_FUNCS_IN('json_object', 'jansson')
if not conf.CONFIG_GET('HAVE_JSON_OBJECT'):
if Options.options.with_json_audit == True:
conf.fatal('JSON support requested, but no suitable jansson '
'library found')
if Options.options.with_json_audit != False:
conf.fatal("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 '

View File

@ -49,8 +49,8 @@
</itemizedlist>
<para>Authentication and authorization audit information is logged
under the auth_audit, and if Samba is compiled against the jansson
JSON library, a JSON representation is logged under
under the auth_audit, and if Samba was not compiled with
--without-json-audit, a JSON representation is logged under
auth_json_audit.</para>
<para>Support is comprehensive for all authentication and authorisation