[Git][NTPsec/ntpsec][master] Fix(#793): "./waf bin_test" results in a Python error

Hal Murray (@hal.murray) gitlab at mg.gitlab.com
Thu Jan 30 21:43:56 UTC 2025



Hal Murray pushed to branch master at NTPsec / ntpsec


Commits:
dc69f9e6 by James Browning at 2025-01-30T21:40:18+00:00
Fix(#793): "./waf bin_test" results in a Python error

- - - - -


1 changed file:

- wscript


Changes:

=====================================
wscript
=====================================
@@ -932,16 +932,10 @@ class check(BuildContext):
     variant = "main"
 
 
-def bin_test(ctx):
+class bin_test(BuildContext):
     """Run binary check, use after tests."""
-    from wafhelpers.bin_test import cmd_bin_test
-    cmd_bin_test(ctx)
-
-
-def bin_test_summary(ctx):
-    """Display results of binary check, use after tests."""
-    from wafhelpers.bin_test import bin_test_summary
-    bin_test_summary(ctx)
+    cmd = 'bin_test'
+    variant = "main"
 
 
 variant_cmd = (
@@ -1133,6 +1127,7 @@ def build(ctx):
     # Skip running unit tests on a cross compile build
     from waflib import Options
     if not ctx.env.ENABLE_CROSS:
+        from wafhelpers.bin_test import cmd_bin_test, bin_test_summary
         # Force re-running of tests.  Same as 'waf --alltests'
         if ctx.cmd == "check":
             ctx.options.all_tests = True
@@ -1144,7 +1139,7 @@ def build(ctx):
             return
 
         # Test binaries
-        ctx.add_post_fun(bin_test)
+        ctx.add_post_fun(cmd_bin_test)
 
         # Write test log to a file
         ctx.add_post_fun(test_write_log)



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

-- 
View it on GitLab: https://gitlab.com/NTPsec/ntpsec/-/commit/dc69f9e6c5a2583036b793bd93939f9cd3bdf665
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/20250130/ce7cd05c/attachment-0001.htm>


More information about the vc mailing list