From: Joe Guo <joeg@catalyst.net.nz>
Date: Tue, 30 Apr 2019 05:21:15 +0000 (+1200)
Subject: wscript: mv --enable-coverage option to global
X-Git-Tag: tdb-1.4.1~83
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=21475e9556e3088a99951abeea07cea500cce105;p=thirdparty%2Fsamba.git

wscript: mv --enable-coverage option to global

We want to enable gcov for all tasks, move it to global wscript.

Signed-off-by: Joe Guo <joeg@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
---

diff --git a/buildtools/wafsamba/wscript b/buildtools/wafsamba/wscript
index f1ebc1cea9e..2e14a326cb8 100644
--- a/buildtools/wafsamba/wscript
+++ b/buildtools/wafsamba/wscript
@@ -102,6 +102,10 @@ def options(opt):
     gr.add_option('--enable-developer',
                    help=("Turn on developer warnings and debugging"),
                    action="store_true", dest='developer', default=False)
+    opt.add_option('--enable-coverage',
+                   help=("enable options necessary for code coverage "
+                         "reporting on selftest (default=no)"),
+                   action="store_true", dest='enable_coverage', default=False)
     def picky_developer_callback(option, opt_str, value, parser):
         parser.values.developer = True
         parser.values.picky_developer = True
@@ -278,6 +282,13 @@ def configure(conf):
     except:
         conf.env.ABI_CHECK = False
 
+    conf.env.enable_coverage = Options.options.enable_coverage
+    if conf.env.enable_coverage:
+        conf.ADD_LDFLAGS('-lgcov', testflags=True)
+        conf.ADD_CFLAGS('--coverage', testflags=True)
+        # disable abi check for coverage, otherwise ld will fail
+        conf.env.ABI_CHECK = False
+
     conf.env.GIT_LOCAL_CHANGES = Options.options.GIT_LOCAL_CHANGES
 
     conf.CHECK_UNAME()
diff --git a/selftest/wscript b/selftest/wscript
index 44455480eb1..5116d7ee31b 100644
--- a/selftest/wscript
+++ b/selftest/wscript
@@ -20,10 +20,6 @@ def options(opt):
     opt.add_option('--enable-selftest',
                    help=("enable options necessary for selftest (default=no)"),
                    action="store_true", dest='enable_selftest', default=False)
-    opt.add_option('--enable-coverage',
-                   help=("enable options necessary for code coverage "
-                         "reporting on selftest (default=no)"),
-                   action="store_true", dest='enable_coverage', default=False)
     opt.add_option('--with-selftest-prefix',
                    help=("specify location of selftest directory "
                          "(default=%s)" % DEFAULT_SELFTEST_PREFIX),
@@ -97,11 +93,6 @@ def options(opt):
 
 def configure(conf):
     conf.env.SELFTEST_PREFIX = Options.options.SELFTEST_PREFIX
-    conf.env.enable_coverage = Options.options.enable_coverage
-    if conf.env.enable_coverage:
-        conf.ADD_LDFLAGS('-lgcov', testflags=True)
-        conf.ADD_CFLAGS('--coverage')
-
     if Options.options.enable_selftest or Options.options.developer:
         conf.DEFINE('ENABLE_SELFTEST', 1)