Skip to content

Commit

Permalink
Fix tools/BUILD.bazel to make 'bazel cquery ...' work
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
moroten authored and EdSchouten committed Dec 20, 2024
1 parent 078d9d7 commit ff8c80c
Showing 1 changed file with 2 additions and 14 deletions.
16 changes: 2 additions & 14 deletions tools/BUILD.bazel
Original file line number Diff line number Diff line change
@@ -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.
Expand Down

0 comments on commit ff8c80c

Please sign in to comment.