-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Android Platforms does not work with AAR Import #21225
Comments
We'll address this in the Starlark version of aar_import in rules_android. |
Since Bazel 7.1.0 is around the corner, I think we can actually get this into the upcoming release. |
Would be great if this was CP'd onto the release! Thanks for the timely fix :) |
@bazel-io fork 7.1.0 |
Fixes bazelbuild#21225 PiperOrigin-RevId: 610551472 Change-Id: I7e62919dd41d0d4e2b923df1942920162f554809
…21502) Fixes #21225 Commit 1e77944 PiperOrigin-RevId: 610551472 Change-Id: I7e62919dd41d0d4e2b923df1942920162f554809 Co-authored-by: Googler <[email protected]>
A fix for this issue has been included in Bazel 7.1.0 RC2. Please test out the release candidate and report any issues as soon as possible. |
…tforms and toolchain resolution. This mirrors the changes in native aar_import: bazelbuild/bazel#21225 PiperOrigin-RevId: 621204597 Change-Id: I75f9eb693b30fc0987a376b877fda156990dc6cf
Description of the bug:
Hi, all.
We recently upgraded to bazel 7.0.2 which according to this post, enabled android platforms for use: https://blog.bazel.build/2023/11/15/android-platforms.html
I am cross compiling android apps with the following bazel rc flags:
I am noticing an issue with a maven dependency:
This command below:
fails to compile with the following error;
It looks like the offending line is the --cpu k8, and indeed when I cd'd into the bazel sandbox and ran the command with --cpu arm64-v8a it ran without error. I tracked down the offending line to:
bazel/src/main/java/com/google/devtools/build/lib/rules/android/AarImport.java
Line 440 in 224d642
My code does work with the following bazelrc and platform mappings flags (e.g we have a workaround)
Platform mappings
We're hoping to get rid of this code soon, so wondering if I am missing something here.
We actually ran into this when we were on a bazel 7 pre-release and reported an issue here: bazelbuild/rules_android_ndk#39
Thanks in advance!
Which category does this issue belong to?
Android
What's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.
On bazel 7, use android platforms, and attempt to compile the maven dependency:
Which operating system are you running Bazel on?
Linux
What is the output of
bazel info release
?release 7.0.2
If
bazel info release
returnsdevelopment version
or(@non-git)
, tell us how you built Bazel.No response
What's the output of
git remote get-url origin; git rev-parse HEAD
?Is this a regression? If yes, please try to identify the Bazel commit where the bug was introduced.
I don't think this is a regression
Have you found anything relevant by searching the web?
We actually ran into this when we were on a bazel 7 pre-release and reported an issue here: bazelbuild/rules_android_ndk#39
Any other information, logs, or outputs that you want to share?
No response
The text was updated successfully, but these errors were encountered: