Treat plugins as host tools.

As a followup to ca3ead7745, treat plugin executables as well as protoc as host tools.
This commit is contained in:
Benjamin Peterson 2019-06-03 08:56:33 -07:00 committed by Adam Cozzette
parent a8b8c037fc
commit 6153f80876

View File

@ -117,6 +117,7 @@ def _proto_gen_impl(ctx):
outs = [ctx.actions.declare_file(out, sibling = src) for out in outs]
inputs = [src] + deps
tools = [ctx.executable.protoc]
if ctx.executable.plugin:
plugin = ctx.executable.plugin
lang = ctx.attr.plugin_language
@ -131,11 +132,12 @@ def _proto_gen_impl(ctx):
outdir = ",".join(ctx.attr.plugin_options) + ":" + outdir
args += [("--plugin=protoc-gen-%s=" + path_tpl) % (lang, plugin.path)]
args += ["--%s_out=%s" % (lang, outdir)]
inputs += [plugin]
tools.append(plugin)
if not in_gen_dir:
ctx.actions.run(
inputs = inputs,
tools = tools,
outputs = outs,
arguments = args + import_flags + [src.path],
executable = ctx.executable.protoc,
@ -162,7 +164,7 @@ def _proto_gen_impl(ctx):
outputs = [out],
command = command,
mnemonic = "ProtoCompile",
tools = [ctx.executable.protoc],
tools = tools,
use_default_shell_env = True,
)