Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

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
Contributor

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
Contributor 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
Contributor 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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants