Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Get core-for-system-modules.jar via AndroidSdkInfo.#477

Open
oscarthecat wants to merge 1 commit into
bazelbuild:main from
oscarthecat:feature/core-for-system-modules-jar-via-sdk-info
Open

Get core-for-system-modules.jar via AndroidSdkInfo. #477
oscarthecat wants to merge 1 commit into
bazelbuild:main from
oscarthecat:feature/core-for-system-modules-jar-via-sdk-info

Conversation

@oscarthecat

@oscarthecat oscarthecat commented Apr 7, 2026
edited
Loading

Copy link
Copy Markdown

tools/jdk/BUILD previously hardcoded @androidsdk//:core-for-system-modules-jar, which assumes a local SDK is always present. Add core_for_system_modules_jar to AndroidSdkInfo and expose it via a toolchain-aware rule so that any SDK toolchain implementation (local or remote) can provide the jar.

google-cla Bot commented Apr 7, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

ahumesky commented Apr 8, 2026

Copy link
Copy Markdown
Collaborator

Could you update your fork to the latest, CI will need at least oscarthecat@3bf8bae to pass

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /