1266263 - Bump minimum Android NDK version to 15 since minimum Android SDK version is 16

Status: RESOLVED WORKSFORME

(Firefox Build System :: Android Studio and Gradle Integration, defect)

Description

[Makoto Kato [:m_kato]]

In http://mxr.mozilla.org/mozilla-central/source/build/autoconf/android.m4, MIN_ANDROID_VERSION is 9.

And mozconfig (http://mxr.mozilla.org/mozilla-central/source/mobile/android/config/mozconfigs/common) still uses "ac_add_options --with-android-version=9"

Since we set "ac_add_options --with-android-min-sdk=15", we should also set --with-android-version=15.

Comment 1

[Richard Newman [:rnewman]]

Last time this came up, I recall that it was more complicated than it seems, and perhaps not valuable to do so. Sebastian might remember.

Comment 2

[Richard Newman [:rnewman]]

See Bug 1133459 Comment 4. We explicitly didn't bump this in Bug 1237342.

Comment 3

[Sebastian Kaspari (:sebastian; :pocmo)]

(In reply to Richard Newman [:rnewman] from comment #1)
> Last time this came up, I recall that it was more complicated than it seems,
> and perhaps not valuable to do so. Sebastian might remember.

I only remember having problems with the target sdk version. If I read the linked bugs correctly --with-android-version is only used for configuring/using the NDK.

@snorp: Do we need to update this?

Comment 4

[James Willcox (:snorp) ([e-mail]) (he/him)]

I don't think there is really any benefit to updating this. The NDK version we use is independent of the SDK, and we've had problems trying to use a newer NDK platform version in the past. We will eventually have to do this for 64-bit support, but right now there is no need.

Comment 5

[Makoto Kato [:m_kato]]

Android NDK r17 removes anroid-9, so we should bump to 16 when supporting NDK r17.

Comment 6

[Nathan Froyd [:froydnj]]

The minimum NDK version is 20 now (minimum Android version is 16 on 32-bit platforms, 21 on 64-bit platforms), so I think we can close this as WORKSFORME.