From 3b54337be54107a5f196bb98dfc858b10ec35b8a Mon Sep 17 00:00:00 2001 From: denglesberg-splunk <125308860+denglesberg-splunk@users.noreply.github.com> Date: Fri, 19 Jan 2024 03:12:54 -0500 Subject: [PATCH] Replace `exec_tools` with `tools` in bazel genrule. (#77510) As of the Bazel 6.x series, there is no difference between the `exec_tools` and `tools`. Bazel 7 removes the `exec_tools` attribute entirely. This commit updates to use the cannonical attribute name to allow building `clang-tidy``with bazel 7.0.0, though it does not change the default bazel version which remains at 6.1.2. See also https://github.com/bazelbuild/bazel/issues/19132 for more information. --- .../clang-tools-extra/clang-tidy/BUILD.bazel | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utils/bazel/llvm-project-overlay/clang-tools-extra/clang-tidy/BUILD.bazel b/utils/bazel/llvm-project-overlay/clang-tools-extra/clang-tidy/BUILD.bazel index bbabc5397e9893..317863de3b36ca 100644 --- a/utils/bazel/llvm-project-overlay/clang-tools-extra/clang-tidy/BUILD.bazel +++ b/utils/bazel/llvm-project-overlay/clang-tools-extra/clang-tidy/BUILD.bazel @@ -57,7 +57,7 @@ genrule( srcs = ["misc/ConfusableTable/confusables.txt"], outs = ["Confusables.inc"], cmd = "$(location :confusable_table_builder) $(SRCS) $(OUTS)", - exec_tools = [":confusable_table_builder"], + tools = [":confusable_table_builder"], visibility = ["//visibility:private"], )