From 3a42560cbcfa5d301562362d99fb7812dd127249 Mon Sep 17 00:00:00 2001
From: William Chargin <wchargin@gmail.com>
Date: Thu, 17 Jan 2019 11:46:40 -0800
Subject: [PATCH] build: lock down visibility of unversioned summary targets
 (#1769)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Summary:
We’ve migrated all users inside Google to use the explicitly versioned
endpoints (either `summary:summary_v1` or `summary:summary_v2`), so we
can now enforce that going forward.

Any existing users should migrate to `//tensorboard/summary:summary_v1`,
which is API-compatible and stable.

Test Plan:
That `bazel build //tensorboard` and `bazel query 'deps(//...)'` still
work suffices.

wchargin-branch: restrict-summary-visibility
---
 tensorboard/BUILD         | 2 +-
 tensorboard/summary/BUILD | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/tensorboard/BUILD b/tensorboard/BUILD
index db4d8621bc..10d20f1c39 100644
--- a/tensorboard/BUILD
+++ b/tensorboard/BUILD
@@ -343,7 +343,7 @@ py_test(
 alias(
     name = "summary",
     actual = "//tensorboard/summary",
-    visibility = ["//visibility:public"],
+    visibility = ["//tensorboard:internal"],
 )
 
 py_library(
diff --git a/tensorboard/summary/BUILD b/tensorboard/summary/BUILD
index 2f109d14fd..e4630576bb 100644
--- a/tensorboard/summary/BUILD
+++ b/tensorboard/summary/BUILD
@@ -9,7 +9,7 @@ py_library(
         "__init__.py",
     ],
     srcs_version = "PY2AND3",
-    visibility = ["//visibility:public"],
+    visibility = ["//tensorboard:internal"],
     deps = [
         ":summary_v1",
         ":summary_v2",