Skip to content

Commit

Permalink
Fix bazel build
Browse files Browse the repository at this point in the history
In recent bazel versions, `exec_tools` attribute has been removed from `genrule`
cf. bazelbuild/bazel#19132
  • Loading branch information
qinghui-xu committed Jun 3, 2024
1 parent 0356eae commit ada5560
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions build_defs/internal_shell.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ def inline_sh_binary(
native.genrule(
name = name + "_genrule",
srcs = srcs,
exec_tools = tools,
tools = tools,
outs = [name + ".sh"],
cmd = "cat <<'EOF' >$(OUTS)\n#!/bin/bash -exu\n%s\nEOF\n" % cmd,
visibility = ["//visibility:private"],
Expand Down Expand Up @@ -77,7 +77,7 @@ def inline_sh_test(
native.genrule(
name = name + "_genrule",
srcs = srcs,
exec_tools = tools,
tools = tools,
outs = [name + ".sh"],
cmd = "cat <<'EOF' >$(OUTS)\n#!/bin/bash -exu\n%s\nEOF\n" % cmd,
visibility = ["//visibility:private"],
Expand Down
2 changes: 1 addition & 1 deletion src/google/protobuf/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,7 @@ genrule(
--proto_path=$$(dirname $$(dirname $$(dirname $(location any.proto)))) \
$(SRCS)
""",
exec_tools = ["//:protoc"],
tools = ["//:protoc"],
visibility = ["//visibility:private"],
)

Expand Down

0 comments on commit ada5560

Please sign in to comment.