Use rules_shell instead of built-in rules

bazelbuild · Dec 12, 2024 · dada973 · dada973
1 parent 9cdb86c
commit dada973
Show file tree

Hide file tree

Showing 20 changed files with 45 additions and 1 deletion.
diff --git a/.bazelrc b/.bazelrc
@@ -60,6 +60,10 @@ build --nolegacy_external_runfiles
 # Required for cargo_build_script support before Bazel 7
 build --incompatible_merge_fixed_and_default_shell_env
 
+# Guarantee that rules_rust does not rely on autoloading.
+# https://github.com/bazelbuild/bazel/issues/23043
+build --incompatible_autoload_externally=
+
 ###############################################################################
 ## Bzlmod
 ###############################################################################

diff --git a/MODULE.bazel b/MODULE.bazel
@@ -29,6 +29,10 @@ bazel_dep(
     name = "rules_license",
     version = "1.0.0",
 )
+bazel_dep(
+    name = "rules_shell",
+    version = "0.3.0",
+)
 bazel_dep(
     name = "apple_support",
     version = "1.17.1",

diff --git a/docs/BUILD.bazel b/docs/BUILD.bazel
@@ -1,4 +1,5 @@
 load("@bazel_skylib//:bzl_library.bzl", "bzl_library")
+load("@rules_shell//shell:sh_binary.bzl", "sh_binary")
 load("@stardoc//stardoc:stardoc.bzl", "stardoc")
 load("//:page.bzl", "gen_header", "page")
 

diff --git a/examples/bazel_env/BUILD.bazel b/examples/bazel_env/BUILD.bazel
@@ -1,5 +1,7 @@
 "Tests for upstream wrappers."
 
+load("@rules_shell//shell:sh_test.bzl", "sh_test")
+
 sh_test(
     name = "upstream_cargo_test",
     size = "small",

diff --git a/examples/bazel_env/MODULE.bazel b/examples/bazel_env/MODULE.bazel
@@ -42,3 +42,8 @@ crate.from_cargo(
 use_repo(crate, "crates")
 
 bazel_dep(name = "bazel_env.bzl", version = "0.1.1")
+
+bazel_dep(
+    name = "rules_shell",
+    version = "0.3.0",
+)
diff --git a/examples/bzlmod/all_crate_deps/BUILD.bazel b/examples/bzlmod/all_crate_deps/BUILD.bazel
@@ -1,6 +1,7 @@
 load("@bazel_skylib//rules:build_test.bzl", "build_test")
 load("@crates//:defs.bzl", "all_crate_deps")
 load("@rules_rust//rust:defs.bzl", "rust_binary", "rust_doc")
+load("@rules_shell//shell:sh_test.bzl", "sh_test")
 
 package(default_visibility = ["//visibility:public"])
 

diff --git a/examples/bzlmod/cross_compile/BUILD.bazel b/examples/bzlmod/cross_compile/BUILD.bazel
@@ -1,4 +1,5 @@
 load("@rules_rust//rust:defs.bzl", "rust_binary")
+load("@rules_shell//shell:sh_test.bzl", "sh_test")
 
 package(default_visibility = ["//visibility:public"])
 

diff --git a/examples/crate_universe_local_path/BUILD.bazel b/examples/crate_universe_local_path/BUILD.bazel
@@ -1,3 +1,5 @@
+load("@rules_shell//shell:sh_binary.bzl", "sh_binary")
+
 sh_binary(
     name = "vendor_lazy_static_out_of_tree",
     srcs = ["vendor_lazy_static.sh"],

diff --git a/examples/ios_build/BUILD.bazel b/examples/ios_build/BUILD.bazel
@@ -1,4 +1,5 @@
 load("@rules_rust//rust:defs.bzl", "rust_static_library")
+load("@rules_shell//shell:sh_test.bzl", "sh_test")
 
 exports_files([
     "Cargo.toml",

diff --git a/examples/musl_cross_compiling/BUILD.bazel b/examples/musl_cross_compiling/BUILD.bazel
@@ -1,6 +1,7 @@
 load("@aspect_bazel_lib//lib:transitions.bzl", "platform_transition_binary")
 load("@bazel_skylib//rules:build_test.bzl", "build_test")
 load("@rules_rust//rust:defs.bzl", "rust_binary")
+load("@rules_shell//shell:sh_test.bzl", "sh_test")
 
 rust_binary(
     name = "hello",

diff --git a/examples/sys/basic/BUILD.bazel b/examples/sys/basic/BUILD.bazel
@@ -13,6 +13,7 @@
 # limitations under the License.
 
 load("@rules_rust//rust:defs.bzl", "rust_binary")
+load("@rules_shell//shell:sh_test.bzl", "sh_test")
 
 package(default_visibility = ["//visibility:public"])
 

diff --git a/rust/private/repository_utils.bzl b/rust/private/repository_utils.bzl
@@ -96,6 +96,8 @@ def BUILD_for_cargo(target_triple):
     )
 
 _build_file_for_rustfmt_template = """\
+load("@rules_shell//shell:sh_binary.bzl", "sh_binary")
+
 filegroup(
     name = "rustfmt_bin",
     srcs = ["bin/rustfmt{binary_ext}"],

diff --git a/rust/repositories.bzl b/rust/repositories.bzl
@@ -85,6 +85,16 @@ def rules_rust_dependencies():
         ],
         sha256 = "26d4021f6898e23b82ef953078389dd49ac2b5618ac564ade4ef87cced147b38",
     )
+    maybe(
+        http_archive,
+        name = "rules_shell",
+        urls = [
+            "https://mirror.bazel.build/github.com/bazelbuild/rules_shell/releases/download/v0.3.0/rules_shell-v0.3.0.tar.gz",
+            "https://github.com/bazelbuild/rules_shell/releases/download/v0.3.0/rules_shell-v0.3.0.tar.gz",
+        ],
+        sha256 = "d8cd4a3a91fc1dc68d4c7d6b655f09def109f7186437e3f50a9b60ab436a0c53",
+        strip_prefix = "rules_shell-0.3.0",
+    )
     maybe(
         http_archive,
         name = "bazel_features",

diff --git a/test/BUILD.bazel b/test/BUILD.bazel
@@ -1,3 +1,5 @@
+load("@rules_shell//shell:sh_binary.bzl", "sh_binary")
+
 sh_binary(
     name = "query_test_binary",
     srcs = ["query_test_binary.sh"],

diff --git a/test/cc_shared_library/BUILD.bazel b/test/cc_shared_library/BUILD.bazel
@@ -1,5 +1,6 @@
 load("@rules_cc//cc:defs.bzl", "cc_import", "cc_library", "cc_test")
 load("@rules_rust//rust:defs.bzl", "rust_binary", "rust_static_library")
+load("@rules_shell//shell:sh_test.bzl", "sh_test")
 
 rust_static_library(
     name = "rust_lib",

diff --git a/test/clippy/BUILD.bazel b/test/clippy/BUILD.bazel
@@ -8,6 +8,7 @@ load(
     "rust_static_library",
     "rust_test",
 )
+load("@rules_shell//shell:sh_binary.bzl", "sh_binary")
 
 package(default_visibility = ["//test:__subpackages__"])
 

diff --git a/test/empty_suite/BUILD.bazel b/test/empty_suite/BUILD.bazel
@@ -1,4 +1,5 @@
 load("//rust:defs.bzl", "rust_library", "rust_test", "rust_test_suite")
+load("@rules_shell//shell:sh_test.bzl", "sh_test")
 
 # This package has a rust_test_suite containing no tests.
 # (This could happen if the srcs were seleceted via a glob).

diff --git a/test/portable_link_flags_darwin/BUILD.bazel b/test/portable_link_flags_darwin/BUILD.bazel
@@ -1,5 +1,6 @@
 load("@rules_cc//cc:defs.bzl", "cc_library")
 load("@rules_rust//rust:defs.bzl", "rust_binary", "rust_library")
+load("@rules_shell//shell:sh_test.bzl", "sh_test")
 
 rust_library(
     name = "rust-lib",

diff --git a/test/rust_analyzer/BUILD.bazel b/test/rust_analyzer/BUILD.bazel
@@ -1,3 +1,5 @@
+load("@rules_shell//shell:sh_binary.bzl", "sh_binary")
+
 sh_binary(
     name = "rust_analyzer_test",
     srcs = ["rust_analyzer_test_runner.sh"],

diff --git a/test/rustfmt/rustfmt_integration_test_suite.bzl b/test/rustfmt/rustfmt_integration_test_suite.bzl
@@ -8,6 +8,7 @@ load(
     "rust_static_library",
     "rustfmt_test",
 )
+load("@rules_shell//shell:sh_binary.bzl", "sh_binary")
 
 _VARIANTS = {
     "rust_binary": rust_binary,
@@ -115,7 +116,7 @@ def rustfmt_integration_test_suite(name, **kwargs):
         **kwargs
     )
 
-    native.sh_binary(
+    sh_binary(
         name = "{}.test_runner".format(name),
         srcs = ["rustfmt_failure_test.sh"],
         testonly = True,