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

Added support for emu x86 #567

Merged
merged 3 commits into from Sep 28, 2018
Merged

Added support for emu x86 #567

merged 3 commits into from Sep 28, 2018

Conversation

keianhzo
Copy link
Collaborator

Added support for android x86 emulator. Ideally we could add support for flavor dimensions and have targets for abi, platforms and channels (release, beta, nightly) so we can easily test all combinations. We can do that in the future.

@bluemarvin
Copy link
Contributor

This change looks like the noApi will now only build for x86? I still use the noApi for testing on arm.

@keianhzo
Copy link
Collaborator Author

@bluemarvin Fixed that and added multi flavor support as it's a more standard way of dealing with this. Haven't added support for channels as I'm not really sure we need that yet and it generates a lot of variants.

@keianhzo keianhzo force-pushed the x86emu-support branch 3 times, most recently from 3fbafb5 to 6d0c1ef Compare September 25, 2018 16:42
@bluemarvin
Copy link
Contributor

this looks great. I wish I could do a test push to master to see if taskcluster has any problems with this before landing it for real.

@bluemarvin
Copy link
Contributor

bluemarvin commented Sep 26, 2018

I can build all the versions of the apk with:

./gradlew --no-daemon clean assembleGooglevr assembleNoapi assembleWavevr assembleSvr assembleOculusvr

But when I try to run noapix86 on an emulator by pressing the green play button in AndroidStudio I get the following error:

taskclusterBuild = true
//geckoViewLocal = /Users/rbarker/Downloads/geckoview-default-armeabi-v7a-64.0.20180921212116.aar
Generated versionCode: 22691351

WARNING: Cannot build selected target ABI: x86, no suitable splits configured: armeabi-v7a;
WARNING: Cannot build selected target ABI: x86, no suitable splits configured: armeabi-v7a;
WARNING: Cannot build selected target ABI: x86, no suitable splits configured: armeabi-v7a;
WARNING: Cannot build selected target ABI: x86, no suitable splits configured: armeabi-v7a;
WARNING: Cannot build selected target ABI: x86, no suitable splits configured: armeabi-v7a;
WARNING: Cannot build selected target ABI: x86, no suitable splits configured: armeabi-v7a;
WARNING: Cannot build selected target ABI: x86, no suitable splits configured: armeabi-v7a;
WARNING: Cannot build selected target ABI: x86, no suitable splits configured: armeabi-v7a;
WARNING: Cannot build selected target ABI: x86, no suitable splits configured: armeabi-v7a;
WARNING: Cannot build selected target ABI: x86, no suitable splits configured: armeabi-v7a;

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring project ':app'.
> Cannot build for ABI: x86 supported ABIs are: armeabi-v7a

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 6s

Copy link
Contributor

@bluemarvin bluemarvin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@keianhzo Can you see if you can fix the play button? I tried poking at it but didn't see anything obvious.

@MortimerGoro
Copy link
Contributor

I have the same problem

Tried to disable splits and update AS/plugins with no luck:

    splits {
        abi {
            enable false
        }
    }

@keianhzo
Copy link
Collaborator Author

Fixed the MacOS ABI filtering issue, should work fine in all platforms now.

@bluemarvin bluemarvin merged commit 82a2d0b into master Sep 28, 2018
@bluemarvin bluemarvin deleted the x86emu-support branch October 9, 2018 19:13
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.

None yet

4 participants