From ff8c80c6eb4c844b4b79f4f015babc10addc1139 Mon Sep 17 00:00:00 2001 From: Fredrik Medley Date: Fri, 22 Nov 2024 10:45:56 +0100 Subject: [PATCH] Fix tools/BUILD.bazel to make 'bazel cquery ...' work ibazel (https://github.com/bazelbuild/bazel-watcher/) is running `bazel cquery ...` which failed when there are executable targets in the deps attribute of a go_library. This was never checked in CI because of the manual tag of the tools target. --- tools/BUILD.bazel | 16 ++-------------- 1 file changed, 2 insertions(+), 14 deletions(-) diff --git a/tools/BUILD.bazel b/tools/BUILD.bazel index c842447c..80ecb9e2 100644 --- a/tools/BUILD.bazel +++ b/tools/BUILD.bazel @@ -1,18 +1,6 @@ -load("@aspect_bazel_lib//lib:expand_template.bzl", "expand_template") -load("@rules_go//go:def.bzl", "go_library") +# gazelle:ignore -go_library( - name = "tools", - srcs = ["deps.go"], - importpath = "github.com/buildbarn/bb-storage/tools", - tags = ["manual"], - visibility = ["//visibility:public"], - deps = [ - "@cc_mvdan_gofumpt//:gofumpt", - "@com_github_bazelbuild_buildtools//buildifier", - "@org_golang_x_lint//:lint", - ], -) +load("@aspect_bazel_lib//lib:expand_template.bzl", "expand_template") # When built with --stamp, creates a non-deterministic output file for pushing images to a remote registry. # With --nostamp, produces a deterministic output so dependents get cache hits.