Merge pull request #2609 from hzeller/feature-20260917-build-clean
Only depend on necessary dependencies.
diff --git a/.github/bin/run-build-cleaner.sh b/.github/bin/run-build-cleaner.sh
index b8d1af5..d5b03f7 100755
--- a/.github/bin/run-build-cleaner.sh
+++ b/.github/bin/run-build-cleaner.sh
@@ -18,6 +18,10 @@
BANT=$($(dirname $0)/get-bant-path.sh)
+# Run build so that we have all dependencies downloaded and genrules
+# materialized.
+bazel build -k --remote_download_outputs=all ...
+
if "${BANT}" -q dwyu ... ; then
echo "Dependencies ok." >&2
else
diff --git a/MODULE.bazel b/MODULE.bazel
index 463c634..724056c 100644
--- a/MODULE.bazel
+++ b/MODULE.bazel
@@ -20,4 +20,4 @@
bazel_dep(name = "googletest", version = "1.17.0.bcr.2", dev_dependency = True)
# To build compilation DB and run build-cleaning
-bazel_dep(name = "bant", version = "0.3.4", dev_dependency = True)
+bazel_dep(name = "bant", version = "0.3.6", dev_dependency = True)
diff --git a/verible/verilog/formatting/BUILD b/verible/verilog/formatting/BUILD
index eb089df..6e4782e 100644
--- a/verible/verilog/formatting/BUILD
+++ b/verible/verilog/formatting/BUILD
@@ -193,77 +193,118 @@
],
)
-_FORMATTER_E2E_TEST_DEPS = [
- ":format-style",
- ":formatter",
- ":formatter-test-utils",
- "//verible/common/formatting:align",
- "//verible/common/formatting:basic-format-style",
- "//verible/common/strings:display-utils",
- "//verible/common/strings:position",
- "//verible/common/text:text-structure",
- "//verible/common/util:interval",
- "//verible/common/util:logging",
- "//verible/verilog/analysis:verilog-analyzer",
- "@abseil-cpp//absl/log:die_if_null",
- "@abseil-cpp//absl/status",
- "@abseil-cpp//absl/strings",
- "@googletest//:gtest",
- "@googletest//:gtest_main",
-]
-
cc_test(
name = "formatter_test",
srcs = ["formatter_test.cc"],
- deps = _FORMATTER_E2E_TEST_DEPS,
+ deps = [
+ ":format-style",
+ ":formatter",
+ ":formatter-test-utils",
+ "//verible/common/text:text-structure",
+ "//verible/common/util:logging",
+ "//verible/verilog/analysis:verilog-analyzer",
+ "@abseil-cpp//absl/log:die_if_null",
+ "@abseil-cpp//absl/status",
+ "@googletest//:gtest",
+ "@googletest//:gtest_main",
+ ],
)
cc_test(
name = "formatter_macro_test",
srcs = ["formatter_macro_test.cc"],
- deps = _FORMATTER_E2E_TEST_DEPS,
+ deps = [
+ ":formatter-test-utils",
+ "@googletest//:gtest",
+ "@googletest//:gtest_main",
+ ],
)
cc_test(
name = "formatter_module_test",
srcs = ["formatter_module_test.cc"],
- deps = _FORMATTER_E2E_TEST_DEPS,
+ deps = [
+ ":formatter-test-utils",
+ "@googletest//:gtest",
+ "@googletest//:gtest_main",
+ ],
)
cc_test(
name = "formatter_class_package_test",
srcs = ["formatter_class_package_test.cc"],
- deps = _FORMATTER_E2E_TEST_DEPS,
+ deps = [
+ ":formatter-test-utils",
+ "@googletest//:gtest",
+ "@googletest//:gtest_main",
+ ],
)
cc_test(
name = "formatter_function_task_test",
srcs = ["formatter_function_task_test.cc"],
- deps = _FORMATTER_E2E_TEST_DEPS,
+ deps = [
+ ":formatter-test-utils",
+ "@googletest//:gtest",
+ "@googletest//:gtest_main",
+ ],
)
cc_test(
name = "formatter_align_test",
srcs = ["formatter_align_test.cc"],
- deps = _FORMATTER_E2E_TEST_DEPS,
+ deps = [
+ ":format-style",
+ ":formatter",
+ ":formatter-test-utils",
+ "//verible/common/formatting:align",
+ "//verible/common/util:logging",
+ "@googletest//:gtest",
+ "@googletest//:gtest_main",
+ ],
)
cc_test(
name = "formatter_wrap_style_test",
srcs = ["formatter_wrap_style_test.cc"],
- deps = _FORMATTER_E2E_TEST_DEPS,
+ deps = [
+ ":format-style",
+ ":formatter",
+ ":formatter-test-utils",
+ "//verible/common/formatting:align",
+ "//verible/common/formatting:basic-format-style",
+ "//verible/common/strings:display-utils",
+ "//verible/common/strings:position",
+ "//verible/common/util:interval",
+ "//verible/common/util:logging",
+ "@abseil-cpp//absl/status",
+ "@abseil-cpp//absl/strings",
+ "@googletest//:gtest",
+ "@googletest//:gtest_main",
+ ],
)
cc_test(
name = "formatter_disable_test",
srcs = ["formatter_disable_test.cc"],
- deps = _FORMATTER_E2E_TEST_DEPS,
+ deps = [
+ ":formatter-test-utils",
+ "@googletest//:gtest",
+ "@googletest//:gtest_main",
+ ],
)
cc_test(
name = "formatter_issue_regression_test",
srcs = ["formatter_issue_regression_test.cc"],
- deps = _FORMATTER_E2E_TEST_DEPS,
+ deps = [
+ ":format-style",
+ ":formatter",
+ ":formatter-test-utils",
+ "//verible/common/util:logging",
+ "@googletest//:gtest",
+ "@googletest//:gtest_main",
+ ],
)
cc_test(