Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add exec constraints to local Java runtime toolchains #113

Merged
merged 1 commit into from
Aug 7, 2023

Conversation

fmeum
Copy link
Contributor

@fmeum fmeum commented Jun 9, 2023

Fixes incorrect behavior in cross-platform build situations where a local Java runtime would end up in the runfiles of a java_binary target built for a platform on which it can't be executed.

Work towards https://github.com/bazelbuild/bazel/issues#18265
Split off from bazelbuild/bazel#18262

@fmeum fmeum requested review from comius and a team as code owners June 9, 2023 07:30
@fmeum
Copy link
Contributor Author

fmeum commented Jun 9, 2023

@comius @hvadehra Could you review this PR?

@fmeum
Copy link
Contributor Author

fmeum commented Jul 5, 2023

@hvadehra Could you also review this? It's a follow-up to #114 and the last part of bazelbuild/bazel#18262 that affects rules_java.

Fixes incorrect behavior in cross-platform build situations where a
local Java runtime would end up in the runfiles of a `java_binary`
target built for a platform on which it can't be executed.

Work towards https://github.com/bazelbuild/bazel/issues#18265
Split off from bazelbuild/bazel#18262
@copybara-service copybara-service bot merged commit 124d5a7 into bazelbuild:master Aug 7, 2023
1 check passed
@fmeum fmeum deleted the 17085-exec-constraints branch August 7, 2023 07:59
@fmeum
Copy link
Contributor Author

fmeum commented Aug 7, 2023

@hvadehra Without #114, this PR in isolation may result in Android compilation failures when Bazel is updated to use a java_tools version. Do you already know whether #114 can be merged soon?

@hvadehra
Copy link
Member

hvadehra commented Aug 7, 2023

Yeah, there was an issue with the import, I'm trying to get it to work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants