[Git][NTPsec/ntpsec][master] I: NTPsec build does not repect --notests

James Browning (@jamesb_fe80) gitlab at mg.gitlab.com
Mon Apr 11 01:18:09 UTC 2022



James Browning pushed to branch master at NTPsec / ntpsec


Commits:
1a7bb2e3 by James Browning at 2022-04-10T16:26:08-07:00
I: NTPsec build does not repect --notests

STR: (run the following)
- git clone https://gitlab.com/NTPsec/ntpsec
- cd ntpsec
- ./waf configure build --notests -p

AR: NTPsec runs tests despite having a parameter telling it not to
ER: NTPsec should not run tests when it has been told not to
- - - - -


1 changed file:

- wscript


Changes:

=====================================
wscript
=====================================
@@ -1131,6 +1131,7 @@ def build(ctx):
     ctx.manpage(8, "ntpclients/ntpsnmpd-man.adoc")
 
     # Skip running unit tests on a cross compile build
+    from waflib import Options
     if not ctx.env.ENABLE_CROSS:
         # Force re-running of tests.  Same as 'waf --alltests'
         if ctx.cmd == "check":
@@ -1139,6 +1140,8 @@ def build(ctx):
             # Print log if -v is supplied
             if verbose > 0:
                 ctx.add_post_fun(test_print_log)
+        elif Options.options.no_tests:
+            return
 
         # Test binaries
         ctx.add_post_fun(bin_test)
@@ -1152,7 +1155,6 @@ def build(ctx):
         ctx.add_post_fun(bin_test_summary)
     else:
         pprint("YELLOW", "Unit test runner skipped on a cross-compiled build.")
-        from waflib import Options
         Options.options.no_tests = True
 
     if ctx.cmd == "build":



View it on GitLab: https://gitlab.com/NTPsec/ntpsec/-/commit/1a7bb2e3a2749bd709ea4cf10b66b6f6d05aaf9d

-- 
View it on GitLab: https://gitlab.com/NTPsec/ntpsec/-/commit/1a7bb2e3a2749bd709ea4cf10b66b6f6d05aaf9d
You're receiving this email because of your account on gitlab.com.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ntpsec.org/pipermail/vc/attachments/20220411/a6837aef/attachment-0001.htm>


More information about the vc mailing list