[cpp] add explicit files for deps

and make `ray_cpp_pkg` private visibility

Signed-off-by: Lonnie Liu <[email protected]>

Author: aslonnie

cpp/BUILD.bazel

@@ -1,6 +1,7 @@
 # Bazel build
 # C/C++ documentation: https://docs.bazel.build/versions/master/be/c-cpp.html
 
+load("//bazel:python.bzl", "py_test_module_list")
 load("//bazel:ray.bzl", "COPTS")
 
 cc_binary(
@@ -56,12 +57,12 @@ cc_library(
     strip_include_prefix = "include",
     visibility = ["//visibility:public"],
     deps = [
-        "//src/ray/gcs/gcs_client:global_state_accessor_lib",
         "//:ray_common",
         "//src/ray/core_worker:core_worker_lib",
+        "//src/ray/gcs/gcs_client:global_state_accessor_lib",
         "//src/ray/util",
-        "//src/ray/util:process",
         "//src/ray/util:cmd_line_utils",
+        "//src/ray/util:process",
         "@boost//:callable_traits",
         "@boost//:dll",
         "@com_google_absl//absl/flags:flag",
@@ -109,6 +110,15 @@ cc_binary(
     }),
 )
 
+filegroup(
+    name = "ray_cpp_pkg_files",
+    srcs = [
+        "default_worker",
+        "libray_api.so",
+    ],
+    visibility = ["//visibility:private"],
+)
+
 genrule(
     name = "ray_cpp_pkg",
     srcs = [
@@ -163,7 +173,7 @@ genrule(
         echo "$$WORK_DIR" > $@
     """,
     local = 1,
-    visibility = ["//visibility:public"],
+    visibility = ["//visibility:private"],
 )
 
 # test
@@ -200,8 +210,8 @@ cc_test(
     data = [
         "counter.so",
         "plus.so",
-        "ray_cpp_pkg",
         "src/ray/test/cluster/test_cross_language_invocation.py",
+        ":ray_cpp_pkg_files",
     ],
     linkstatic = True,
     tags = ["team:core"],
@@ -223,7 +233,7 @@ cc_test(
     ],
     copts = COPTS,
     data = [
-        "ray_cpp_pkg",
+        ":ray_cpp_pkg_files",
         "//java:libio_ray_ray_test.jar",
     ],
     linkstatic = True,
@@ -392,8 +402,6 @@ cc_binary(
     ],
 )
 
-load("//bazel:python.bzl", "py_test_module_list")
-
 py_test_module_list(
     size = "medium",
     extra_srcs = [],

python/setup.py

@@ -528,7 +528,8 @@ def build(build_python, build_java, build_cpp):
         )
         raise OSError(msg)
 
-    bazel_env = dict(os.environ, PYTHON3_BIN_PATH=sys.executable)
+    bazel_env = dict(os.environ)
+    bazel_env["PYTHON3_BIN_PATH"] = sys.executable
 
     if is_native_windows_or_msys():
         SHELL = bazel_env.get("SHELL")