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

[Android] Choose appropriate ABI when running depending on target device #409

Closed
jgavris opened this issue Sep 8, 2018 · 3 comments
Closed
Labels
lang: c++ C++ rules integration P4 This is either out of scope or we don't have bandwidth to review a PR. (No assignee) product: Android Studio Android Studio plugin type: feature request

Comments

@jgavris
Copy link

jgavris commented Sep 8, 2018

For Android applications with native dependencies (cc_library etc), compiling each architecture can be quite expensive (arm64-v8a for device and x86_64 for simulator).

Our current workflow either requires us to specify --fat_apk_cpus=arm64-v8a,x86_64 ahead of time and spend twice as long to build, or go in and change the configuration before each build.

Would it be possible to behave more like Xcode's 'build only active architecture' where the target architecture is derived from the device to be run on?

@brendandouglas brendandouglas added the product: Android Studio Android Studio plugin label Sep 18, 2018
@vsiva
Copy link
Collaborator

vsiva commented Oct 31, 2018

Does mobile install not do this optimization?

@jgavris
Copy link
Author

jgavris commented Oct 31, 2018

@vsiva mobile-install is currently broken for Android 9 (see bazelbuild/examples#77). I don't believe it does, either. I've heard rumors that mobile-install is being replaced with a v2 version so no further efforts are being made to fix the current version.

@mai93 mai93 added type: feature request lang: c++ C++ rules integration labels Feb 10, 2021
@sgowroji sgowroji added the awaiting-maintainer Awaiting review from Bazel team on issues label Nov 18, 2022
@mai93 mai93 added the P4 This is either out of scope or we don't have bandwidth to review a PR. (No assignee) label Dec 5, 2022
@mai93 mai93 removed their assignment Dec 5, 2022
@mai93 mai93 removed the awaiting-maintainer Awaiting review from Bazel team on issues label Dec 5, 2022
@sgowroji
Copy link
Member

Hi there! We're doing a clean up of old issues and will be closing this one. Please reopen if you’d like to discuss anything further. We’ll respond as soon as we have the bandwidth/resources to do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang: c++ C++ rules integration P4 This is either out of scope or we don't have bandwidth to review a PR. (No assignee) product: Android Studio Android Studio plugin type: feature request
Projects
None yet
Development

No branches or pull requests

5 participants