1187974 - ANDROID_NDK_VERSION doesn't do anything, gets out of date

Status: RESOLVED DUPLICATE of bug 1207680

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

Description

[Gian-Carlo Pascutto [:gcp]]

https://wiki.mozilla.org/Mobile/Fennec/Android#Android_NDK_and_SDK_version_notes

claims the NDK version needed to build Fennec is indicated here:
http://hg.mozilla.org/mozilla-central/annotate/tip/mobile/android/config/mozconfigs/common#l32

But this isn't actually used for the builders, and currently even outdated.

There's zero warnings if you're trying to build with an unsupported NDK and the only place it's indicated is a Wiki page that is observable (see above) out of date.

Comment 1

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

I think Dylan expressed an interest in build systems at one point :)

Comment 2

[Mike Hommey [:glandium]]

All our build environments are x86_64, now, so we can remove the uname checks from all our mozconfigs (for desktop too). That would remove the branch that uses ANDROID_NDK_VERSION, at which point we can remove ANDROID_NDK_VERSION.

Comment 3

[Nick Alexander :nalexander [he/him]]

Post Bug 1204260, this is gone.  But see Bug 1207680, which tracks extracting versions pins to a central location and consuming them throughout.