diff options
author | Florian Weikert <fwe@google.com> | 2016-10-11 11:30:04 +0000 |
---|---|---|
committer | Yue Gan <yueg@google.com> | 2016-10-11 13:26:43 +0000 |
commit | e23b79c31ccaba176515e7032553d2816e5832cd (patch) | |
tree | 11845142e7bd61ed1a13f9b23f463b4f3a3c2522 /tools | |
parent | 8d36a34ee02ad0fd07d713b8c8ee273ff30d2fb9 (diff) |
Only add the runfiles of artifacts that are actual inputs of the spawn action.
The previous solution added too many unneeded runfiles, which caused problems with our workers.
--
MOS_MIGRATED_REVID=135782773
Diffstat (limited to 'tools')
-rw-r--r-- | tools/build_rules/genproto.bzl | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/tools/build_rules/genproto.bzl b/tools/build_rules/genproto.bzl index b067f12672..8270387c4e 100644 --- a/tools/build_rules/genproto.bzl +++ b/tools/build_rules/genproto.bzl @@ -31,7 +31,8 @@ def gensrcjar_impl(ctx): ctx.executable._gensrcjar.path, ]), inputs=([ctx.file.src] + ctx.files._gensrcjar + ctx.files._jar + - ctx.files._jdk + ctx.files._proto_compiler), + ctx.files._jdk + ctx.files._proto_compiler + + ctx.files.grpc_java_plugin), outputs=[out], mnemonic="GenProtoSrcJar", use_default_shell_env=True) @@ -51,7 +52,7 @@ gensrcjar = rule( single_file = True, ), "_gensrcjar": attr.label( - default = Label(str(Label("//tools/build_rules:gensrcjar"))), + default = Label("//tools/build_rules:gensrcjar"), cfg = "host", executable = True, ), |