Move Cargo binaries to //cargo/private (#3694)

This is to clarify that the interface of these binaries are internal.
They're only use is to power actions for other `//cargo` rules.

Author: UebelAndre

.bazelci/presubmit.yml

@@ -62,7 +62,7 @@ default_rbe_targets: &default_rbe_targets
   - "//..."
   # These require the upstream tools wrapper to include a stdlib, which doesn't currently work on RBE.
   # We should fix this. See https://github.com/bazelbuild/rules_rust/issues/3252
-  - "-//cargo/cargo_toml_variable_extractor/testdata/..."
+  - "-//cargo/private/cargo_toml_variable_extractor/testdata/..."
 default_macos_targets: &default_macos_targets
   - "--"
   - "//..."

cargo/3rdparty/crates/defs.bzl

@@ -293,100 +293,100 @@ def aliases(
 ###############################################################################
 
 _NORMAL_DEPENDENCIES = {
-    "cargo/cargo_toml_variable_extractor": {
+    "cargo/private/cargo_toml_info": {
+        _COMMON_CONDITION: {
+            "cargo_toml": Label("@rrc//:cargo_toml-0.20.5"),
+        },
+    },
+    "cargo/private/cargo_toml_variable_extractor": {
         _COMMON_CONDITION: {
             "cargo-util-schemas": Label("@rrc//:cargo-util-schemas-0.3.1"),
             "pathdiff": Label("@rrc//:pathdiff-0.1.0"),
             "semver": Label("@rrc//:semver-1.0.25"),
             "toml": Label("@rrc//:toml-0.8.20"),
         },
     },
-    "cargo/private/cargo_toml_info": {
-        _COMMON_CONDITION: {
-            "cargo_toml": Label("@rrc//:cargo_toml-0.20.5"),
-        },
-    },
 }
 
 _NORMAL_ALIASES = {
-    "cargo/cargo_toml_variable_extractor": {
+    "cargo/private/cargo_toml_info": {
         _COMMON_CONDITION: {
         },
     },
-    "cargo/private/cargo_toml_info": {
+    "cargo/private/cargo_toml_variable_extractor": {
         _COMMON_CONDITION: {
         },
     },
 }
 
 _NORMAL_DEV_DEPENDENCIES = {
-    "cargo/cargo_toml_variable_extractor": {
-    },
     "cargo/private/cargo_toml_info": {
     },
+    "cargo/private/cargo_toml_variable_extractor": {
+    },
 }
 
 _NORMAL_DEV_ALIASES = {
-    "cargo/cargo_toml_variable_extractor": {
-    },
     "cargo/private/cargo_toml_info": {
     },
+    "cargo/private/cargo_toml_variable_extractor": {
+    },
 }
 
 _PROC_MACRO_DEPENDENCIES = {
-    "cargo/cargo_toml_variable_extractor": {
-    },
     "cargo/private/cargo_toml_info": {
     },
+    "cargo/private/cargo_toml_variable_extractor": {
+    },
 }
 
 _PROC_MACRO_ALIASES = {
-    "cargo/cargo_toml_variable_extractor": {
-    },
     "cargo/private/cargo_toml_info": {
     },
+    "cargo/private/cargo_toml_variable_extractor": {
+    },
 }
 
 _PROC_MACRO_DEV_DEPENDENCIES = {
-    "cargo/cargo_toml_variable_extractor": {
-    },
     "cargo/private/cargo_toml_info": {
     },
+    "cargo/private/cargo_toml_variable_extractor": {
+    },
 }
 
 _PROC_MACRO_DEV_ALIASES = {
-    "cargo/cargo_toml_variable_extractor": {
-    },
     "cargo/private/cargo_toml_info": {
     },
+    "cargo/private/cargo_toml_variable_extractor": {
+    },
 }
 
 _BUILD_DEPENDENCIES = {
-    "cargo/cargo_toml_variable_extractor": {
-    },
     "cargo/private/cargo_toml_info": {
     },
+    "cargo/private/cargo_toml_variable_extractor": {
+    },
 }
 
 _BUILD_ALIASES = {
-    "cargo/cargo_toml_variable_extractor": {
-    },
     "cargo/private/cargo_toml_info": {
     },
+    "cargo/private/cargo_toml_variable_extractor": {
+    },
 }
 
 _BUILD_PROC_MACRO_DEPENDENCIES = {
-    "cargo/cargo_toml_variable_extractor": {
-    },
     "cargo/private/cargo_toml_info": {
     },
+    "cargo/private/cargo_toml_variable_extractor": {
+    },
 }
 
 _BUILD_PROC_MACRO_ALIASES = {
-    "cargo/cargo_toml_variable_extractor": {
-    },
     "cargo/private/cargo_toml_info": {
     },
+    "cargo/private/cargo_toml_variable_extractor": {
+    },
 }
 
 _CONDITIONS = {

cargo/Cargo.toml

@@ -1,5 +1,5 @@
 [workspace]
 members = [
-    "cargo_toml_variable_extractor",
+    "private/cargo_toml_variable_extractor",
     "private/cargo_toml_info",
 ]

cargo/bootstrap/BUILD.bazel renamed to cargo/private/bootstrap/BUILD.bazel

@@ -136,7 +136,7 @@ rust_binary(
         "{binary}",
     ],
     srcs = [
-        "@rules_rust//cargo/bootstrap:bootstrap_installer.rs"
+        "@rules_rust//cargo/private/bootstrap:bootstrap_installer.rs"
     ],
 )
 """

cargo/bootstrap/bootstrap_installer.rs renamed to cargo/private/bootstrap/bootstrap_installer.rs

@@ -710,7 +710,7 @@ cargo_build_script = rule(
         "_cargo_build_script_runner": attr.label(
             executable = True,
             allow_files = True,
-            default = Label("//cargo/cargo_build_script_runner:runner"),
+            default = Label("//cargo/private/cargo_build_script_runner:runner"),
             cfg = "exec",
         ),
         "_cargo_manifest_dir_filename_suffixes_to_retain": attr.label(