-
Notifications
You must be signed in to change notification settings - Fork 1.3k
[android] - update getting started and distribution docs #8412
Conversation
@tobrun, thanks for your PR! By analyzing this pull request, we identified @tmcw, @tmpsantos and @jfirebaugh to be potential reviewers. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@@ -1,21 +1,30 @@ | |||
# Distributing Mapbox GL Native for Android | |||
|
|||
Depending on your use case, you may want to support all or just a subset of [Android ABIs](http://developer.android.com/ndk/guides/abis.html). | |||
This can be achieved using the different `Makefile` targets that are available. | |||
Depending on your use case, you may want to support all or just a subset of [Android ABIs](http://developer.android.com/ndk/guides/abis.html). This document covers building an `.aar` file from the Mapbox Android SDK and building `.so` files for specific ABIs. In normal circumstances an application developer will use [APK splits](https://developer.android.com/studio/build/configure-apk-splits.html) to optimize this at application level. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tobrun It'd make sense to link https://github.com/mapbox/mapbox-gl-native/wiki/Strategies-to-reduce-Android-APK-size here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added to #8390 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tobrun Thanks.
refs #8390, this PR updates the Getting started page, installation instructions and expands the distributing documentation. The requirements on the old pages didn't reflect the current state. Requires #8363 to be merged first.