….create_compilation_action` (bazelbuild#30981) This allows faster iteration on the `rules_java` <> `JavaBuilder` contract without having to wait for Bazel changes/releases. In the immediate future, this will be used for the unused-deps checking. Longer term this should also help in migrating off the native compilation code. Note: the added integration shell test is currently a no-op on CI (since we need the corresponding rules_java changes) but does allow for local testing with an overridden `@rules_java`. Closes bazelbuild#30808. PiperOrigin-RevId: 971826913 Change-Id: I8047e057716475a7a4d8bff48b8c7bfe6d79ba1c (cherry picked from commit 0e6f09d) 9.3.0 adaptation: `java_common.bzl` has no `_ALLOWLIST` constant on this branch, so `bazel_internal/test_rules` is added to the allowlist passed inline to `check_private_api`. `JavaCompileActionBuilderTest` needed a `testutil:TestConstants` dep for the new test. The `bazel_java_test.sh` cases are left out: they exercise `--experimental_check_unused_deps`, which is a JavaBuilder option added by bazelbuild#30807 and not present on this branch, and they are a no-op on CI anyway. `JavaCompileActionBuilderTest` and all four shards of `//src/test/shell/bazel:bazel_java_test_jdk17_toolchain_head` pass locally. Closes bazelbuild#30968 Co-authored-by: Jonathan Perry <jonathan.perry@gs.com>

Uh oh!

[9.3.0] Bazel compilation changes for unused dependencies checking #30968

Description

Activity

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions