diff options
author | Julio Merino <jmmv@google.com> | 2016-02-17 15:32:47 +0000 |
---|---|---|
committer | Kristina Chodorow <kchodorow@google.com> | 2016-02-17 19:37:43 +0000 |
commit | 806d9c6ad28d549112937dd1c261baac29574b98 (patch) | |
tree | b7c2a4c578c9d4f0f17004b8ce060291c2c43133 | |
parent | 1bab6b10e72f9c2877af51cc6193c3dad3be6d33 (diff) |
Reformat genproto.bzl to comply with the style guide.
This is a preparatory change for a follow-up commit so that the diff of that
commit is easy to review.
--
MOS_MIGRATED_REVID=114857369
-rw-r--r-- | tools/build_rules/genproto.bzl | 42 |
1 files changed, 24 insertions, 18 deletions
diff --git a/tools/build_rules/genproto.bzl b/tools/build_rules/genproto.bzl index b93f8b8e16..17177dda5f 100644 --- a/tools/build_rules/genproto.bzl +++ b/tools/build_rules/genproto.bzl @@ -38,24 +38,30 @@ def gensrcjar_impl(ctx): use_default_shell_env = True) gensrcjar = rule( - gensrcjar_impl, - attrs={ - "src": attr.label(allow_files=proto_filetype, single_file=True), - # TODO(bazel-team): this should be a hidden attribute with a default - # value, but Skylark needs to support select first. - "_proto_compiler": attr.label( - default=Label("@bazel_tools//third_party:protoc"), - allow_files=True, - single_file=True), - "_jar": attr.label( - default=Label("@bazel_tools//tools/jdk:jar"), - allow_files=True, - single_file=True), - "_jdk": attr.label( - default=Label("@bazel_tools//tools/jdk:jdk"), - allow_files=True), - }, - outputs={"srcjar": "lib%{name}.srcjar"}, + gensrcjar_impl, + attrs = { + "src": attr.label( + allow_files = proto_filetype, + single_file = True, + ), + # TODO(bazel-team): this should be a hidden attribute with a default + # value, but Skylark needs to support select first. + "_proto_compiler": attr.label( + default = Label("@bazel_tools//third_party:protoc"), + allow_files = True, + single_file = True, + ), + "_jar": attr.label( + default = Label("@bazel_tools//tools/jdk:jar"), + allow_files = True, + single_file = True, + ), + "_jdk": attr.label( + default = Label("@bazel_tools//tools/jdk:jdk"), + allow_files = True, + ), + }, + outputs = {"srcjar": "lib%{name}.srcjar"}, ) # TODO(bazel-team): support proto => proto dependencies too |