Use more explicit visibilities.

RELNOTES: None.
PiperOrigin-RevId: 229371245

BUILD

@@ -1,5 +1,3 @@
-package(default_visibility = ["//visibility:public"])
-
 licenses(["notice"])
 
 exports_files(["LICENSE"])
@@ -14,4 +12,8 @@ filegroup(
         "@build_bazel_apple_support//rules:for_bazel_tests",
         "@build_bazel_apple_support//tools:for_bazel_tests",
     ],
+    # Exposed publicly just so other rules can use this if they set up
+    # integration tests that need to copy all the support files into
+    # a temporary workspace for the tests.
+    visibility = ["//visibility:public"],
 )

lib/BUILD

@@ -1,10 +1,9 @@
 licenses(["notice"])
 
-package(default_visibility = ["//visibility:public"])
-
 # Consumed by bazel tests.
 filegroup(
     name = "for_bazel_tests",
     testonly = 1,
     srcs = glob(["**"]),
+    visibility = ["//:__pkg__"],
 )

rules/BUILD

@@ -1,10 +1,9 @@
 licenses(["notice"])
 
-package(default_visibility = ["//visibility:public"])
-
 # Consumed by bazel tests.
 filegroup(
     name = "for_bazel_tests",
     testonly = 1,
     srcs = glob(["**"]),
+    visibility = ["//:__pkg__"],
 )

tools/BUILD

@@ -1,19 +1,24 @@
 licenses(["notice"])
 
-package(default_visibility = ["//visibility:public"])
-
 # Additional files, such as tools, that should be present in a test's runfiles
 # during coverage collection.
-filegroup(name = "coverage_support")
+filegroup(
+    name = "coverage_support",
+    # Public so other rules can add the dependencies via this group.
+    visibility = ["//visibility:public"],
+)
 
 sh_binary(
     name = "xcode_path_wrapper",
     srcs = ["xcode_path_wrapper.sh"],
+    # Added to rules as a private attribute, so it needs to be public.
+    visibility = ["//visibility:public"],
 )
 
 # Consumed by bazel tests.
 filegroup(
     name = "for_bazel_tests",
     testonly = 1,
     srcs = glob(["**"]),
+    visibility = ["//:__pkg__"],
 )