commit b7cae414a1642de72217c646366da7c39ffb33b1
author Edwin Kempin <ekempin@google.com> Thu Jan 07 17:14:18 2021 +0100
committer Edwin Kempin <ekempin@google.com> Mon Jan 11 15:19:24 2021 +0100
tree 350421069f413262d96147cc997b9a28c8794358
parent f72b4af837455f9b314ab88398153170a6354397

Restrict the visibility of the proto target

Only expose it to the packages that need it.

Also-by: David Ostrovsky <david@ostrovsky.org>
Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: I0cf1caf15a860ddba227d10ab48eac14ab99135c

proto/BUILD

diff --git a/proto/BUILD b/proto/BUILD
index 05494e4..ed0f86f 100644
--- a/proto/BUILD
+++ b/proto/BUILD
@@ -1,8 +1,6 @@
 load("@rules_proto//proto:defs.bzl", "proto_library")
 load("@rules_java//java:defs.bzl", "java_proto_library")
 
-package(default_visibility = ["//plugins/code-owners:visibility"])
-
 proto_library(
     name = "owners_metadata_proto",
     srcs = ["owners_metadata.proto"],
@@ -10,6 +8,10 @@
 
 java_proto_library(
     name = "owners_metadata_java_proto",
-    visibility = ["//visibility:public"],
+    visibility = [
+        "//plugins/code-owners/java/com/google/gerrit/plugins/codeowners/api:__pkg__",
+        "//plugins/code-owners/java/com/google/gerrit/plugins/codeowners/backend:__pkg__",
+        "//plugins/code-owners/java/com/google/gerrit/plugins/codeowners/restapi:__pkg__",
+    ],
     deps = [":owners_metadata_proto"],
 )