[PATCH] build: add option to choose to build with or without JSON audit support
douglas.bagnall at catalyst.net.nz
Wed Apr 18 05:51:34 UTC 2018
On 18/04/18 17:24, Stefan Metzmacher via samba-technical wrote:
>> Is there some way of meeting those concerns in this patch? Would
>> squashing this in be acceptable?:
> Isn't this part of Björn's patch enough?
> + 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')
Ah, OK, sorry. Thanks for pointing this out Metze.
>> That lets you run make test without jansson but means autobuild
>> requires it.
> We don't want make test without jansson, we want to build some packages
> without json-audit support, even if jansson is installed an the system.
Right. I am curious -- does the dependency cause problems on some platforms?
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 500 bytes
Desc: OpenPGP digital signature
More information about the samba-technical