[SCM] Samba Shared Repository - branch master updated
Alexander Bokovoy
ab at samba.org
Fri Sep 7 14:37:42 UTC 2018
On pe, 07 syys 2018, Andrew Bartlett wrote:
> The branch, master has been updated
> via 4847daf wafsamba/samba_waf18: redefine flex function
> via 364077d lib/replace/wscript: fix detection of a fallthrough attribute for clang
> via 311e1eb wafsamba/samba_autoconf: when setting undefined result, use empty tuple
> via 193fdbf s3/wscript: fix flex detection
> via e761271 s3/wscript: fix bison detection
> from 4a63ab9 s4/selftest/tests: Enabled samba.tests.samba_tool.computer
>
> https://git.samba.org/?p=samba.git;a=shortlog;h=master
While Andrew pushed the current patch set, Metze pointed out that it
didn't solve the issue with CONFIG_SET() still reporting undefined
defines are set if they were stored in the cache as '0'.
Attached patch should fix the issue. On FreeBSD 11.2 I now have:
$ egrep '(HAVE_BSD_STRTOLL|INCOHERENT_MMAP|WORKING_STRPTIME)' bin/default/include/config.h bin/c4che/default_cache.py
bin/default/include/config.h:#define HAVE_BSD_STRTOLL 1
bin/default/include/config.h:/* #undef HAVE_WORKING_STRPTIME */
bin/default/include/config.h:/* #undef HAVE_INCOHERENT_MMAP */
bin/c4che/default_cache.py:HAVE_BSD_STRTOLL = 1
bin/c4che/default_cache.py:HAVE_INCOHERENT_MMAP = ()
bin/c4che/default_cache.py:HAVE_WORKING_STRPTIME = ()
--
/ Alexander Bokovoy
-------------- next part --------------
>From 92f2b9b2bac513412169de959285c35a40ea9d07 Mon Sep 17 00:00:00 2001
From: Alexander Bokovoy <ab at samba.org>
Date: Fri, 7 Sep 2018 14:34:48 +0000
Subject: [PATCH] buildtools/wafsamba: undefine a define for a failed test
unless asked to keep it
If conf.CHECK_CODE() is called without `always=True` and the test has
failed, undefine the define already set to '0' by conf.check_code().
This restores expectations that undefined symbols are not considered to
be set by CONFIG_SET() method.
Signed-off-by: Alexander Bokovoy <ab at samba.org>
---
buildtools/wafsamba/samba_autoconf.py | 2 ++
1 file changed, 2 insertions(+)
diff --git a/buildtools/wafsamba/samba_autoconf.py b/buildtools/wafsamba/samba_autoconf.py
index 6b940e53c00..ced824d593d 100644
--- a/buildtools/wafsamba/samba_autoconf.py
+++ b/buildtools/wafsamba/samba_autoconf.py
@@ -447,6 +447,8 @@ def CHECK_CODE(conf, code, define,
except Exception:
if always:
conf.DEFINE(define, 0)
+ else:
+ conf.undefine(define)
conf.COMPOUND_END(False)
if mandatory:
raise
--
2.18.0
More information about the samba-technical
mailing list