-
Notifications
You must be signed in to change notification settings - Fork 4.4k
android_local_test isn't compatible with Android Platforms #19829
Copy link
Copy link
Closed
Labels
P1I'll work on this now. (Assignee required)I'll work on this now. (Assignee required)team-AndroidIssues for Android teamIssues for Android teamtype: bug
Description
Forked from #18262.
The android_local_test rule is a strange beast which wants to do two things: it wants to build an APK, using a valid Android SDK.
But, it also wants to run a Java binary on the local platform, with that APK as a data file. This worked before #18262, but now that we are being honest about Java versions, the cracks show up.
This sounds like a case for execution groups to me, but I also don't actually understand how this works under the hood.
For now, this means that we need to disable testing android_local_test with Android Platforms, and get the code cleaned up.
We probably also need to fix the Starlark version in bazelbuild/rules_android.
Reactions are currently unavailable
Metadata
Metadata
Labels
P1I'll work on this now. (Assignee required)I'll work on this now. (Assignee required)team-AndroidIssues for Android teamIssues for Android teamtype: bug