[SCM] Samba Shared Repository - branch master updated
Andrew Bartlett
abartlet at samba.org
Wed Sep 26 21:41:02 UTC 2018
The branch, master has been updated
via 5a73f90 py3 kcc tests: mark verify test as flapping
via c98f997 turn --with-json-audit into global --with-json
from cb8e61b docs: Only build vfs manpages if the module is enabled
https://git.samba.org/?p=samba.git;a=shortlog;h=master
- Log -----------------------------------------------------------------
commit 5a73f904e192c44e304850287ac439d0b52f7be5
Author: Douglas Bagnall <douglas.bagnall at catalyst.net.nz>
Date: Thu Sep 20 13:10:09 2018 +1200
py3 kcc tests: mark verify test as flapping
Signed-off-by: Douglas Bagnall <douglas.bagnall at catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet at samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet at samba.org>
Autobuild-Date(master): Wed Sep 26 23:40:58 CEST 2018 on sn-devel-144
commit c98f9971922342cbcfd2f55ca076835ba2305c63
Author: Philipp Gesang <philipp.gesang at intra2net.com>
Date: Tue Sep 25 14:09:18 2018 +0200
turn --with-json-audit into global --with-json
Fold the build option --with-json-audit into the toplevel wscript
to reflect the fact that JSON support is no longer local to the
audit subsystem.
Signed-off-by: Philipp Gesang <philipp.gesang at intra2net.com>
Reviewed-by: Douglas Bagnall <douglas.bagnall at catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet at samba.org>
-----------------------------------------------------------------------
Summary of changes:
docs-xml/smbdotconf/logging/loglevel.xml | 2 +-
lib/audit_logging/wscript | 36 --------------------------------
script/autobuild.py | 2 +-
selftest/flapping.d/kcc_verify_py3 | 1 +
wscript | 31 +++++++++++++++++++++++++--
5 files changed, 32 insertions(+), 40 deletions(-)
delete mode 100644 lib/audit_logging/wscript
create mode 100644 selftest/flapping.d/kcc_verify_py3
Changeset truncated at 500 lines:
diff --git a/docs-xml/smbdotconf/logging/loglevel.xml b/docs-xml/smbdotconf/logging/loglevel.xml
index 9a02926..4291272 100644
--- a/docs-xml/smbdotconf/logging/loglevel.xml
+++ b/docs-xml/smbdotconf/logging/loglevel.xml
@@ -56,7 +56,7 @@
<para>Authentication and authorization audit information is logged
under the auth_audit, and if Samba was not compiled with
- --without-json-audit, a JSON representation is logged under
+ --without-json, a JSON representation is logged under
auth_json_audit.</para>
<para>Support is comprehensive for all authentication and authorisation
diff --git a/lib/audit_logging/wscript b/lib/audit_logging/wscript
deleted file mode 100644
index f99df05..0000000
--- a/lib/audit_logging/wscript
+++ /dev/null
@@ -1,36 +0,0 @@
-#!/usr/bin/env python
-
-from waflib import Logs, Options, Errors
-import samba3
-
-def options(opt):
- 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))
-
- return
-
-def configure(conf):
- conf.SET_TARGET_TYPE('jansson', 'EMPTY')
-
- if Options.options.with_json_audit != False:
- if conf.CHECK_CFG(package='jansson', args='--cflags --libs',
- msg='Checking for jansson'):
- conf.CHECK_FUNCS_IN('json_object', 'jansson')
-
- if not conf.CONFIG_GET('HAVE_JSON_OBJECT'):
- if Options.options.with_json_audit != False:
- 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 and the AD DC")
- if not Options.options.without_ad_dc:
- raise Errors.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")
-
diff --git a/script/autobuild.py b/script/autobuild.py
index 02fff41..1dbdf8c 100755
--- a/script/autobuild.py
+++ b/script/autobuild.py
@@ -135,7 +135,7 @@ tasks = {
# We split out this so the isolated ad_dc tests do not wait for ad_dc_ntvfs tests (which are long)
"samba-fileserver": [("random-sleep", "script/random-sleep.sh 60 600", "text/plain"),
- ("configure", "./configure.developer --without-ad-dc --without-ldap --without-ads --without-json-audit --with-selftest-prefix=./bin/ab" + samba_configure_params, "text/plain"),
+ ("configure", "./configure.developer --without-ad-dc --without-ldap --without-ads --without-json --with-selftest-prefix=./bin/ab" + samba_configure_params, "text/plain"),
("make", "make -j", "text/plain"),
("test", "make test FAIL_IMMEDIATELY=1 "
"TESTS='${PY3_ONLY}"
diff --git a/selftest/flapping.d/kcc_verify_py3 b/selftest/flapping.d/kcc_verify_py3
new file mode 100644
index 0000000..03005fc
--- /dev/null
+++ b/selftest/flapping.d/kcc_verify_py3
@@ -0,0 +1 @@
+samba.tests.kcc.python3.samba.tests.kcc.KCCTests.test_verify
diff --git a/wscript b/wscript
index 155de7c..ae1b929 100644
--- a/wscript
+++ b/wscript
@@ -35,7 +35,6 @@ def system_mitkrb5_callback(option, opt, value, parser):
def options(opt):
opt.BUILTIN_DEFAULT('NONE')
opt.PRIVATE_EXTENSION_DEFAULT('samba4')
- opt.RECURSE('lib/audit_logging')
opt.RECURSE('lib/replace')
opt.RECURSE('dynconfig')
opt.RECURSE('packaging')
@@ -103,6 +102,13 @@ def options(opt):
opt.load('python') # options for disabling pyc or pyo compilation
# enable options related to building python extensions
+ opt.add_option('--with-json',
+ action='store_true', dest='with_json',
+ help=("Build with JSON support (default=True). This "
+ "requires the jansson development headers."))
+ opt.add_option('--without-json',
+ action='store_false', dest='with_json',
+ help=("Build without JSON support."))
def configure(conf):
version = samba_version.load_version(env=conf.env)
@@ -234,7 +240,6 @@ def configure(conf):
# system-provided or embedded Heimdal build
if conf.CONFIG_GET('KRB5_VENDOR') in (None, 'heimdal'):
conf.RECURSE('source4/heimdal_build')
- conf.RECURSE('lib/audit_logging')
conf.RECURSE('source4/lib/tls')
conf.RECURSE('source4/dsdb/samdb/ldb_modules')
conf.RECURSE('source4/ntvfs/sysdep')
@@ -274,6 +279,28 @@ def configure(conf):
Logs.warn("pthreadpool support cannot be enabled when __thread support was not found")
conf.undefine('WITH_PTHREADPOOL')
+ conf.SET_TARGET_TYPE('jansson', 'EMPTY')
+
+ if Options.options.with_json != False:
+ if conf.CHECK_CFG(package='jansson', args='--cflags --libs',
+ msg='Checking for jansson'):
+ conf.CHECK_FUNCS_IN('json_object', 'jansson')
+
+ if not conf.CONFIG_GET('HAVE_JSON_OBJECT'):
+ if Options.options.with_json != False:
+ conf.fatal("Jansson JSON support not found. "
+ "Try installing libjansson-dev or jansson-devel. "
+ "Otherwise, use --without-json to build without "
+ "JSON support. "
+ "JSON support is required for the JSON "
+ "formatted audit log feature, the AD DC, and "
+ "the JSON printers of the net utility")
+ if not Options.options.without_ad_dc:
+ raise Errors.WafError('--without-json requires --without-ad-dc. '
+ 'Jansson JSON library is required for '
+ 'building the AD DC')
+ Logs.info("Building without Jansson JSON log support")
+
conf.RECURSE('source3')
conf.RECURSE('lib/texpect')
conf.RECURSE('python')
--
Samba Shared Repository
More information about the samba-cvs
mailing list