]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
Check meson pairwise testing switches
authorAndoni Duarte Pintado <andoni@isc.org>
Mon, 30 Jun 2025 11:14:33 +0000 (13:14 +0200)
committerAndoni Duarte <andoni@isc.org>
Wed, 2 Jul 2025 08:50:34 +0000 (08:50 +0000)
dangerfile.py checked for new configure switches in `configure.ac`,
these were annotated with "# [pairwise:..." in a leading line. Meson
reads those from `meson_options.txt` instead.

dangerfile.py

index b6851be7a574df5510cec473f01e3d66969d7b8a..2dc20bc4c8dd009afe7ec64889a388adbca4e5af 100644 (file)
@@ -406,27 +406,17 @@ if mr_title_action == "sec" and (mr_title_cve is None or "CVE-20" not in mr_titl
 # PAIRWISE TESTING
 ###############################################################################
 #
-# FAIL if the merge request adds any new ./configure switch without an
-# associated annotation used for pairwise testing.
-
-configure_added_lines = added_lines(target_branch, ["configure.ac"])
-switches_added = lines_containing(
-    configure_added_lines, "AC_ARG_ENABLE"
-) + lines_containing(configure_added_lines, "AC_ARG_WITH")
-annotations_added = lines_containing(configure_added_lines, "# [pairwise: ")
+# MESSAGE about pairwise-testing any new build options before merging.
+
+meson_added_lines = added_lines(target_branch, ["meson_options.txt"])
+switches_added = lines_containing(meson_added_lines, "option(")
 if switches_added:
-    if len(switches_added) > len(annotations_added):
-        fail(
-            "This merge request adds at least one new `./configure` switch that "
-            "is not annotated for pairwise testing purposes."
-        )
-    else:
-        message(
-            "**Before merging**, please start a full CI pipeline for this "
-            "branch with the `PAIRWISE_TESTING` variable set to any "
-            "non-empty value (e.g. `1`). This will cause the `pairwise` "
-            "job to exercise the new `./configure` switches."
-        )
+    message(
+        "**Before merging**, please start a full CI pipeline for this "
+        "branch with the `PAIRWISE_TESTING` variable set to any "
+        "non-empty value (e.g. `1`). This will cause the `pairwise` "
+        "job to exercise the new build options added by this merge request."
+    )
 
 ###############################################################################
 # PRE-RELEASE TESTING