Skip to content

Commit

Permalink
bazel: Don't use artifact() for protobuf deps
Browse files Browse the repository at this point in the history
We don't include protobuf in IO_GRPC_GRPC_JAVA_ARTIFACTS, so there might
not actually be an alias available for it to @com_google_protobuf. While
we could add it, it is easier to use the @com_google_protobuf references
directly.

This was preventing `bazel query 'deps(//...)' from succeeding, because
it couldn't find javalite.
  • Loading branch information
ejona86 committed Jul 24, 2024
1 parent 7a25e68 commit d3c2f5a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions protobuf-lite/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@ java_library(
artifact("com.google.guava:guava"),
artifact("com.google.j2objc:j2objc-annotations"),
] + select({
":android": [artifact("com.google.protobuf:protobuf-javalite")],
"//conditions:default": [artifact("com.google.protobuf:protobuf-java")],
":android": ["@com_google_protobuf//:protobuf_javalite"],
"//conditions:default": ["@com_google_protobuf//:protobuf_java"],
}),
)

Expand Down

0 comments on commit d3c2f5a

Please sign in to comment.