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: support fat/multi-arch builds #186

Merged
merged 8 commits into from Jun 12, 2019

Conversation

@mortend
Copy link
Member

commented Jun 12, 2019

This enables building any combination of the following architectures.

  • armeabi-v7a
  • arm64-v8a
  • x86
  • x86_64

Architectures can be specified in the project file, for Debug and Release separately.

"Android": {
  "Architectures": {
    "Debug": ["arm64-v8a"],
    "Release": ["armeabi-v7a", "arm64-v8a"]
  }
}

By default we'll build arm64-v8a for debug, and armeabi-v7a + arm64-v8a for release. Building multiple architectures increases build time significantly, so while testing/developing we usually only want to build one.

Related: #154, fuse-open/fuselibs#1274

mortend added some commits Jun 7, 2019

C++: disable NativeStackTrace on Android
Gives linker error on arm64-v8a.
add project properties for Android.Architecture.{Debug, Release}
Architectures can be specified in the project file, for Debug and Release
separately.

  "Android": {
    "Architectures": {
      "Debug": ["arm64-v8a"],
      "Release": ["armeabi-v7a", "arm64-v8a"]
    }
  }
@kusma

kusma approved these changes Jun 12, 2019

Copy link
Member

left a comment

Looks good to me 🎉

@kusma kusma merged commit a860c7b into fuse-open:master Jun 12, 2019

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@mortend mortend deleted the mortend:fat-android branch Jun 12, 2019

mortend added a commit to mortend/uno that referenced this pull request Aug 21, 2019

Android: revert default ABI fuse-open#214
Some users have reported issues when running Android apps after upgrading to
Uno v1.12, very likely because their devices don't support arm64-v8a, which
is the new default ABI for debug builds.

This seems like a regression, and we should probably have kept armeabi-v7a as
the default ABI in fuse-open#186 for better compatibility.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.