Bump rules_java to 7.12.2

mbland · mbland · commit 20147eb77a44 · 2024-12-03T11:23:27.000-05:00
Updates almost all dependencies on `@bazel_tools//tools/jdk:` toolchain targets and `.bzl` files to corresponding targets and files in `@rules_java//toolchains:`. Part of bazel-contrib#1482 and bazel-contrib#1652. All dependencies on `@bazel_tools//tools/jdk:toolchain_type` remain, however, as there is not yet a corresponding target in `@rules_java//toolchains:`. Adds the `WORKSPACE` stanza recommended in the `rules_java` release notes, and removes our own calls to instantiate `rules_java` repos and to register `rules_java` toolchains. This commit also removes `test/toolchains/jdk.bzl` or `//test/toolchains:java21_toolchain` since we no longer need them. (I actually think we didn't need them with 7.9.0, either, but this seems like a good point at which to remove them.) These changes are required to avoid build breakages when updating to `rules_scala` 7.10.0 and higher. All targets build and all tests pass under Bazel 6.5.0 and 7.4.1. --- I was on the right track in my analysis from bazel-contrib#1619 ("Bump to rules_java 7.9.0 for Bazel 7 compatibility" in the message for commit cd22d88). However, I thought we _shouldn't_ update any targets from `@bazel_tools//tools/jdk:` to `@rules_java//toolchains:`. That's why I thought we were stuck on `rules_java` 7.9.0. However, this comment from @fmeum made me think switching to `@rules_java//toolchains:` actually is the preferred approach: - bazelbuild/rules_java#214 (comment) So this is a potentially breaking change, but in the good kind of way, in that it requires an easy, future proof update.
diff --git a/WORKSPACE b/WORKSPACE
@@ -5,6 +5,12 @@ load("//scala:deps.bzl", "rules_scala_dependencies")
 
 rules_scala_dependencies()
 
+load("@rules_java//java:repositories.bzl", "rules_java_dependencies", "rules_java_toolchains")
+
+rules_java_dependencies()
+
+rules_java_toolchains()
+
 load("@bazel_skylib//:workspace.bzl", "bazel_skylib_workspace")
 
 bazel_skylib_workspace()
@@ -109,17 +115,3 @@ rbe_preconfig(
 load("//scala/private/extensions:dev_deps.bzl", "dev_deps_repositories")
 
 dev_deps_repositories()
-
-# Copied from bazel_tools/tools/jdk/remote_java_repository.bzl.
-[
-    register_toolchains(
-        "@remotejdk21_" + platform + "_toolchain_config_repo//:all",
-    )
-    for platform in [
-        "linux",
-        "macos",
-        "win",
-    ]
-]
-
-register_toolchains("//test/toolchains:java21_toolchain_definition")
diff --git a/jmh/jmh.bzl b/jmh/jmh.bzl
@@ -44,7 +44,7 @@ scala_generate_benchmark = rule(
             ),
         ),
         "runtime_jdk": attr.label(
-            default = Label("@bazel_tools//tools/jdk:current_java_runtime"),
+            default = Label("@rules_java//toolchains:current_java_runtime"),
             providers = [java_common.JavaRuntimeInfo],
         ),
     },
@@ -65,7 +65,10 @@ def scala_benchmark_jmh(**kw):
     testonly = kw.get("testonly", False)
     scalacopts = kw.get("scalacopts", [])
     main_class = kw.get("main_class", "org.openjdk.jmh.Main")
-    runtime_jdk = kw.get("runtime_jdk", "@bazel_tools//tools/jdk:current_java_runtime")
+    runtime_jdk = kw.get(
+        "runtime_jdk",
+        "@rules_java//toolchains:current_java_runtime",
+    )
 
     scala_library(
         name = lib,
diff --git a/scala/deps.bzl b/scala/deps.bzl
@@ -36,9 +36,9 @@ def rules_scala_dependencies():
         http_archive,
         name = "rules_java",
         urls = [
-            "https://github.com/bazelbuild/rules_java/releases/download/7.9.0/rules_java-7.9.0.tar.gz",
+            "https://github.com/bazelbuild/rules_java/releases/download/7.12.2/rules_java-7.12.2.tar.gz",
         ],
-        sha256 = "41131de4417de70b9597e6ebd515168ed0ba843a325dc54a81b92d7af9a7b3ea",
+        sha256 = "a9690bc00c538246880d5c83c233e4deb83fe885f54c21bb445eb8116a180b83",
     )
 
     maybe(
diff --git a/scala/private/common_attributes.bzl b/scala/private/common_attributes.bzl
@@ -29,7 +29,7 @@ common_attrs_for_plugin_bootstrapping = {
     "resource_strip_prefix": attr.string(),
     "resource_jars": attr.label_list(allow_files = True),
     "java_compile_toolchain": attr.label(
-        default = Label("@bazel_tools//tools/jdk:current_java_toolchain"),
+        default = Label("@rules_java//toolchains:current_java_toolchain"),
         providers = [java_common.JavaToolchainInfo],
     ),
     "scalacopts": attr.string_list(),
@@ -82,10 +82,10 @@ common_attrs.update({
 
 implicit_deps = {
     "_java_runtime": attr.label(
-        default = Label("@bazel_tools//tools/jdk:current_java_runtime"),
+        default = Label("@rules_java//toolchains:current_java_runtime"),
     ),
     "_java_host_runtime": attr.label(
-        default = Label("@bazel_tools//tools/jdk:current_host_java_runtime"),
+        default = Label("@rules_java//toolchains:current_host_java_runtime"),
     ),
     "_scalac": attr.label(
         executable = True,
diff --git a/scala/private/extensions/dev_deps.bzl b/scala/private/extensions/dev_deps.bzl
@@ -2,10 +2,8 @@
 
 load("//scala:scala_cross_version.bzl", "default_maven_server_urls")
 load("//scala:scala_maven_import_external.bzl", "java_import_external")
-load("//test/toolchains:jdk.bzl", "remote_jdk21_repositories")
 load("//third_party/repositories:repositories.bzl", "repositories")
 load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")
-load("@rules_java//java:repositories.bzl", "remote_jdk8_repos")
 
 _BUILD_TOOLS_RELEASE = "5.1.0"
 
@@ -45,10 +43,6 @@ def dev_deps_repositories(
         testonly_ = True,
     )
 
-    # We need to select based on platform when we use these
-    # https://github.com/bazelbuild/bazel/issues/11655
-    remote_jdk8_repos()
-
     repositories(
         fetch_sources = fetch_sources,
         for_artifact_ids = [
@@ -73,5 +67,3 @@ def dev_deps_repositories(
         ],
         maven_servers = maven_servers,
     )
-
-    remote_jdk21_repositories()
diff --git a/scala/private/rule_impls.bzl b/scala/private/rule_impls.bzl
@@ -14,7 +14,7 @@
 """Rules for supporting the Scala language."""
 
 load("@bazel_skylib//lib:paths.bzl", "paths")
-load("@bazel_tools//tools/jdk:toolchain_utils.bzl", "find_java_toolchain")
+load("@rules_java//toolchains:toolchain_utils.bzl", "find_java_toolchain")
 load(":common.bzl", "rlocationpath_from_rootpath", _collect_plugin_paths = "collect_plugin_paths")
 load(":resources.bzl", _resource_paths = "paths")
 
diff --git a/scala/private/rules/scala_binary.bzl b/scala/private/rules/scala_binary.bzl
@@ -58,7 +58,7 @@ _scala_binary_attrs = {
     "classpath_resources": attr.label_list(allow_files = True),
     "jvm_flags": attr.string_list(),
     "runtime_jdk": attr.label(
-        default = Label("@bazel_tools//tools/jdk:current_java_runtime"),
+        default = Label("@rules_java//toolchains:current_java_runtime"),
         providers = [java_common.JavaRuntimeInfo],
     ),
 }
diff --git a/scala/private/rules/scala_junit_test.bzl b/scala/private/rules/scala_junit_test.bzl
@@ -79,7 +79,7 @@ _scala_junit_test_attrs = {
     ),
     "jvm_flags": attr.string_list(),
     "runtime_jdk": attr.label(
-        default = Label("@bazel_tools//tools/jdk:current_java_runtime"),
+        default = Label("@rules_java//toolchains:current_java_runtime"),
         providers = [java_common.JavaRuntimeInfo],
     ),
     "env": attr.string_dict(default = {}),
diff --git a/scala/scala_import.bzl b/scala/scala_import.bzl
@@ -145,7 +145,7 @@ scala_import = rule(
             default = Label("//scala/settings:stamp_scala_import"),
         ),
         "java_compile_toolchain": attr.label(
-            default = Label("@bazel_tools//tools/jdk:current_java_toolchain"),
+            default = Label("@rules_java//toolchains:current_java_toolchain"),
         ),
     },
     toolchains = ["@bazel_tools//tools/jdk:toolchain_type"],
diff --git a/scala/scalafmt/phase_scalafmt_ext.bzl b/scala/scalafmt/phase_scalafmt_ext.bzl
@@ -24,7 +24,9 @@ ext_scalafmt = {
             executable = True,
         ),
         "_java_host_runtime": attr.label(
-            default = Label("@bazel_tools//tools/jdk:current_host_java_runtime"),
+            default = Label(
+                "@rules_java//toolchains:current_host_java_runtime",
+            ),
         ),
         "_runner": attr.label(
             allow_single_file = True,
diff --git a/scala_proto/private/scala_proto_aspect.bzl b/scala_proto/private/scala_proto_aspect.bzl
@@ -205,10 +205,12 @@ def _scala_proto_aspect_impl(target, ctx):
 def make_scala_proto_aspect(*extras):
     attrs = {
         "_java_toolchain": attr.label(
-            default = "@bazel_tools//tools/jdk:current_java_toolchain",
+            default = Label("@rules_java//toolchains:current_java_toolchain"),
         ),
         "_java_host_runtime": attr.label(
-            default = "@bazel_tools//tools/jdk:current_host_java_runtime",
+            default = Label(
+                "@rules_java//toolchains:current_host_java_runtime",
+            ),
         ),
     }
     return aspect(
diff --git a/test/BUILD b/test/BUILD
@@ -488,7 +488,7 @@ scala_binary(
     name = "scala_binary_jdk_11",
     srcs = ["ScalaBinaryJdk11.scala"],
     main_class = "scalarules.test.ScalaBinaryJdk11",
-    runtime_jdk = "@bazel_tools//tools/jdk:remote_jdk11",
+    runtime_jdk = "@rules_java//toolchains:remote_jdk11",
 )
 
 # Make sure scala_library respects java_compile_toolchain during builds
@@ -790,14 +790,14 @@ scala_library(
         "src/main/scala/scalarules/test/junit/runtime_platform/JunitRuntimePlatformTest.java",
     ],
     # make sure java compilation toolchain matches runtime toolchain ie --target
-    java_compile_toolchain = "@bazel_tools//tools/jdk:toolchain_java11",
+    java_compile_toolchain = "@rules_java//toolchains:toolchain_java11",
     deps = _JUNIT_DEPS,
 )
 
 scala_junit_test(
     name = "JunitRuntimePlatform_test_runner",
     size = "small",
-    runtime_jdk = "@bazel_tools//tools/jdk:remote_jdk11",
+    runtime_jdk = "@rules_java//toolchains:remote_jdk11",
     suffixes = ["Test"],
     tests_from = [":JunitRuntimePlatform"],
     runtime_deps = [":JunitRuntimePlatform"],
diff --git a/test/jmh/BUILD b/test/jmh/BUILD
@@ -63,7 +63,7 @@ scala_benchmark_jmh(
 scala_benchmark_jmh(
     name = "test_jmh_jdk11",
     srcs = ["TestJmhRuntimeJdk11.scala"],
-    runtime_jdk = "@bazel_tools//tools/jdk:remote_jdk11",
+    runtime_jdk = "@rules_java//toolchains:remote_jdk11",
 )
 
 [sh_test(
diff --git a/test/shell/test_twitter_scrooge.sh b/test/shell/test_twitter_scrooge.sh
@@ -7,10 +7,10 @@ runner=$(get_test_runner "${1:-local}")
 scrooge_compile_with_jdk_11() {
     # javabase and java_toolchain parameters are deprecated and may be
     # removed in Bazel >= 5.0.0
-    bazel build --javabase=@bazel_tools//tools/jdk:remote_jdk11 \
-        --host_javabase=@bazel_tools//tools/jdk:remote_jdk11 \
-        --host_java_toolchain=@bazel_tools//tools/jdk:toolchain_java11 \
-        --java_toolchain=@bazel_tools//tools/jdk:toolchain_java11 \
+    bazel build --javabase=@rules_java//toolchains:remote_jdk11 \
+        --host_javabase=@rules_java//toolchains:remote_jdk11 \
+        --host_java_toolchain=@rules_java//toolchains:toolchain_java11 \
+        --java_toolchain=@rules_java//toolchains:toolchain_java11 \
         --javacopt='--release 11' \
         --java_language_version=11 \
         --tool_java_language_version=11 \
diff --git a/test/src/main/resources/java_sources/BUILD b/test/src/main/resources/java_sources/BUILD
@@ -5,11 +5,11 @@ package(default_visibility = ["//visibility:public"])
 scala_library(
     name = "CompiledWithJava8",
     srcs = ["SimpleJavaSourceFileA.java"],
-    java_compile_toolchain = "@bazel_tools//tools/jdk:toolchain_java8",
+    java_compile_toolchain = "@rules_java//toolchains:toolchain_java8",
 )
 
 scala_library(
     name = "CompiledWithJava11",
     srcs = ["SimpleJavaSourceFileB.java"],
-    java_compile_toolchain = "@bazel_tools//tools/jdk:toolchain_java11",
+    java_compile_toolchain = "@rules_java//toolchains:toolchain_java11",
 )
diff --git a/test/toolchains/BUILD.bazel b/test/toolchains/BUILD.bazel
@@ -1,26 +1,4 @@
 load("//scala:scala_toolchain.bzl", "scala_toolchain")
-load(
-    "@bazel_tools//tools/jdk:default_java_toolchain.bzl",
-    "BASE_JDK9_JVM_OPTS",
-    "DEFAULT_JAVACOPTS",
-    "DEFAULT_TOOLCHAIN_CONFIGURATION",
-    "default_java_toolchain",
-)
-
-default_java_toolchain(
-    name = "java21_toolchain",
-    configuration = DEFAULT_TOOLCHAIN_CONFIGURATION,
-    java_runtime = select({
-        "@platforms//os:linux": "@remotejdk21_linux//:jdk",
-        "@platforms//os:macos": "@remotejdk21_macos//:jdk",
-        "@platforms//os:windows": "@remotejdk21_win//:jdk",
-    }),
-    javacopts = DEFAULT_JAVACOPTS,
-    jvm_opts = BASE_JDK9_JVM_OPTS,
-    source_version = "21",
-    target_version = "21",
-    visibility = ["//visibility:public"],
-)
 
 scala_toolchain(
     name = "ast_plus_one_deps_unused_deps_warn_impl",
diff --git a/test/toolchains/jdk.bzl b/test/toolchains/jdk.bzl
diff --git a/twitter_scrooge/twitter_scrooge.bzl b/twitter_scrooge/twitter_scrooge.bzl
@@ -315,7 +315,9 @@ common_attrs = {
             ),
         ],
     ),
-    "_java_host_runtime": attr.label(default = Label("@bazel_tools//tools/jdk:current_host_java_runtime")),
+    "_java_host_runtime": attr.label(
+        default = Label("@rules_java//toolchains:current_host_java_runtime"),
+    ),
 }
 
 common_aspect_providers = [
@@ -355,7 +357,9 @@ scrooge_java_aspect = aspect(
     attrs = dicts.add(
         common_attrs,
         {
-            "_java_toolchain": attr.label(default = Label("@bazel_tools//tools/jdk:current_java_toolchain")),
+            "_java_toolchain": attr.label(default = Label(
+                "@rules_java//toolchains:current_java_toolchain",
+            )),
         },
     ),
     provides = [ScroogeAspectInfo],

Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ _scala_binary_attrs = {`
`58`	`58`	`"classpath_resources": attr.label_list(allow_files = True),`
`59`	`59`	`"jvm_flags": attr.string_list(),`
`60`	`60`	`"runtime_jdk": attr.label(`
`61`		`- default = Label("@bazel_tools//tools/jdk:current_java_runtime"),`
	`61`	`+ default = Label("@rules_java//toolchains:current_java_runtime"),`
`62`	`62`	`providers = [java_common.JavaRuntimeInfo],`
`63`	`63`	`),`
`64`	`64`	`}`