build: Simplify check for building with ctdb

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Michael Adam <obnox@samba.org>

diff --git a/source3/wscript b/source3/wscript
index 01c9c351421e708c4c93a277914c8d84a207f7d5..191e96b77018b01c0ea6da0a8f9ac7216ad3dc4c 100644 (file)
--- a/source3/wscript
+++ b/source3/wscript
@@ -1460,19 +1460,14 @@ main() {
             conf.DEFINE('WITH_QUOTAS', '1')
 
     #
-    # checking for clustering extensions (CTDB)
+    # checking for clustering (CTDB)
     #
-    if Options.options.with_cluster_support == False:
+    if not Options.options.with_cluster_support:
         # configure is called with --without-cluster-support,
         # so don't check for and build w/o ctdb support.
-        have_cluster_support = False
-
+        Logs.info("building without cluster support (--without-cluster-support)")
+        conf.env.with_ctdb = False
     else:
-
-        have_cluster_support = True
-        ctdb_broken = ""
-        CTDB_CFLAGS = ""
-
         srcdir = os.path.realpath(conf.srcdir)
         if 'EXTRA_INCLUDES' in conf.env:
             includes = ' '.join(conf.env['EXTRA_INCLUDES']).replace('#', srcdir + '/')
@@ -1488,20 +1483,10 @@ main() {
         if not conf.env.USING_SYSTEM_TALLOC:
             includes = includes + ' ' + srcdir + '/lib/talloc'
 
-    if have_cluster_support:
         Logs.info("building with cluster support")
         conf.env['CTDB_CFLAGS'] = '-DCLUSTER_SUPPORT=1'
         conf.env['CTDB_INCLUDE'] = CTDB_INCLUDE
-    else:
-        if Options.options.with_cluster_support == False:
-            Logs.info("building without cluster support (--without-cluster-support)")
-        elif Options.options.with_cluster_support == True:
-            Logs.error("Cluster support not available: " + ctdb_broken)
-            conf.fatal("Cluster support not found, but --with-cluster-support was specified")
-        else:
-            Logs.info("building without cluster support: " + ctdb_broken)
-        conf.env['CTDB_CFLAGS'] = ''
-
+        conf.env.with_ctdb = True
 
     conf.CHECK_CODE('__attribute__((destructor)) static void cleanup(void) { }',
                     'HAVE_FUNCTION_ATTRIBUTE_DESTRUCTOR',

diff --git a/wscript b/wscript
index ee0b7eeca08fc87b7fc3f0deba5e877d07a2a20d..37a1f10a5f2f4c7b2d46d86e49a4e1e99dd65dba 100644 (file)
--- a/wscript
+++ b/wscript
@@ -167,8 +167,7 @@ def configure(conf):
     conf.RECURSE('selftest')
     conf.RECURSE('source3')
     conf.RECURSE('lib/texpect')
-    if Options.options.with_cluster_support:
-        conf.env.with_ctdb = True
+    if conf.env.with_ctdb:
         conf.RECURSE('ctdb')
 
     conf.SAMBA_CHECK_UNDEFINED_SYMBOL_FLAGS()