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

Simplify Android build in preparation for flagless builds #807

Merged
merged 3 commits into from
Aug 2, 2023

Conversation

fmeum
Copy link
Contributor

@fmeum fmeum commented Aug 1, 2023

No description provided.

@fmeum fmeum mentioned this pull request Aug 1, 2023
`__ANDROID__` is automatically defined by the Android NDK, which means
that we no longer need a conditional `--copt`.
Executable `java_binary` targets, i.e., those that can be `bazel run` or
used in build actions, require a Java runtime for the target platform.
Since there is no standalone Java runtime for Android, all Java targets
in the transitive closure of an `android_binary` must not be executable
for cross-compilation to work.

Note: Cross-compilation without flags is currently blocked on
bazelbuild/bazel#17085.
@fmeum fmeum requested a review from a team August 1, 2023 15:24
@fmeum fmeum merged commit 4d0c74b into main Aug 2, 2023
12 checks passed
@fmeum fmeum deleted the FUZZ-741-android-build branch August 2, 2023 10:05
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