Change the default visibility for packages to //visibility:private and then change individual build rules back to their prior visibility. The motivation for this change is making sure future additions are private by default so that new rules are not accidentally public. Some of the rules that are currently public were not meant to be, and the fact that they are marked public does not mean they should be used. As always, the rule is never spell out anything with the word internal (or any synonym for internal). Some of these accidentally public rules will be manually corrected with future changes. PiperOrigin-RevId: 941074322 Change-Id: Ifa046b926a3aec39594bdcf9ed9f70b32a861ba0
diff --git a/absl/log/BUILD.bazel b/absl/log/BUILD.bazel index b1cff4b..f44b1c0 100644 --- a/absl/log/BUILD.bazel +++ b/absl/log/BUILD.bazel
@@ -24,7 +24,7 @@ ) package( - default_visibility = ["//visibility:public"], + default_visibility = ["//visibility:private"], features = [ "header_modules", "layering_check", @@ -41,6 +41,7 @@ hdrs = ["absl_check.h"], copts = ABSL_DEFAULT_COPTS, linkopts = ABSL_DEFAULT_LINKOPTS, + visibility = ["//visibility:public"], deps = [ "//absl/log/internal:check_impl", ], @@ -51,6 +52,7 @@ hdrs = ["absl_log.h"], copts = ABSL_DEFAULT_COPTS, linkopts = ABSL_DEFAULT_LINKOPTS, + visibility = ["//visibility:public"], deps = [ "//absl/log/internal:log_impl", ], @@ -61,6 +63,7 @@ hdrs = ["check.h"], copts = ABSL_DEFAULT_COPTS, linkopts = ABSL_DEFAULT_LINKOPTS, + visibility = ["//visibility:public"], deps = [ "//absl/log/internal:check_impl", "//absl/log/internal:check_op", @@ -76,6 +79,7 @@ hdrs = ["die_if_null.h"], copts = ABSL_DEFAULT_COPTS, linkopts = ABSL_DEFAULT_LINKOPTS, + visibility = ["//visibility:public"], deps = [ ":log", "//absl/base:config", @@ -148,6 +152,7 @@ hdrs = ["log.h"], copts = ABSL_DEFAULT_COPTS, linkopts = ABSL_DEFAULT_LINKOPTS, + visibility = ["//visibility:public"], deps = [ ":vlog_is_on", "//absl/log/internal:log_impl", @@ -160,6 +165,7 @@ hdrs = ["log_entry.h"], copts = ABSL_DEFAULT_COPTS, linkopts = ABSL_DEFAULT_LINKOPTS, + visibility = ["//visibility:public"], deps = [ "//absl/base:config", "//absl/base:core_headers", @@ -178,6 +184,7 @@ hdrs = ["log_sink.h"], copts = ABSL_DEFAULT_COPTS, linkopts = ABSL_DEFAULT_LINKOPTS, + visibility = ["//visibility:public"], deps = [ ":log_entry", "//absl/base:config", @@ -189,6 +196,7 @@ hdrs = ["log_sink_registry.h"], copts = ABSL_DEFAULT_COPTS, linkopts = ABSL_DEFAULT_LINKOPTS, + visibility = ["//visibility:public"], deps = [ ":log_sink", "//absl/base:config", @@ -202,6 +210,7 @@ hdrs = ["log_streamer.h"], copts = ABSL_DEFAULT_COPTS, linkopts = ABSL_DEFAULT_LINKOPTS, + visibility = ["//visibility:public"], deps = [ ":absl_log", "//absl/base:config", @@ -221,6 +230,7 @@ hdrs = ["scoped_mock_log.h"], copts = ABSL_DEFAULT_COPTS, linkopts = ABSL_DEFAULT_LINKOPTS, + visibility = ["//visibility:public"], deps = [ ":log_entry", ":log_sink", @@ -238,6 +248,7 @@ hdrs = ["structured.h"], copts = ABSL_DEFAULT_COPTS, linkopts = ABSL_DEFAULT_LINKOPTS, + visibility = ["//visibility:public"], deps = [ "//absl/base:config", "//absl/base:core_headers", @@ -251,6 +262,7 @@ hdrs = ["absl_vlog_is_on.h"], copts = ABSL_DEFAULT_COPTS, linkopts = ABSL_DEFAULT_LINKOPTS, + visibility = ["//visibility:public"], deps = [ "//absl/base:config", "//absl/base:core_headers", @@ -264,6 +276,7 @@ hdrs = ["vlog_is_on.h"], copts = ABSL_DEFAULT_COPTS, linkopts = ABSL_DEFAULT_LINKOPTS, + visibility = ["//visibility:public"], deps = [ ":absl_vlog_is_on", ],