1194365 - update the android NDK we bootstrap with

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Nathan Froyd [:froydnj]]

We have used NDK r10e for a couple months now.

Comment 1

[Nathan Froyd [:froydnj]]

Attachment: update the android NDK we bootstrap with

I spent a little while today trying to figure out what was going on between
what |mach bootstrap| gave me, what our android mozconfigs say, and what we
actually build with.  Let's fix the bootstrap part of this.

Comment 2

[Nathan Froyd [:froydnj]]

Attachment: update the android NDK we bootstrap with

Here, let's get all the instances of r8e.

Comment 3

[Nick Alexander :nalexander [he/him]]

Comment on attachment 8647684 [details] [diff] [review]
update the android NDK we bootstrap with

Sadly, this doesn't do the trick.  You'll find those URLs are 404; Google now packages a self-extracting executable.  (Bonkers, but hey!  It's Google.)  I am not able to find archives on the web.

I have WIP for using the the new .bin format which I will push.

Comment 4

[Nick Alexander :nalexander [he/him]]

froydn: I pushed a review-without-reviewers to https://reviewboard.mozilla.org/r/16139/, so the commits should be available.

I'd like to make all platforms (Mac and Linux) use the .bin approach.  I think my code above works, I just haven't tested it.  I'd love to see somebody (you!) push it across the line.

Comment 5

[Nathan Froyd [:froydnj]]

(In reply to Nick Alexander :nalexander from comment #4)
> I'd like to make all platforms (Mac and Linux) use the .bin approach.  I
> think my code above works, I just haven't tested it.  I'd love to see
> somebody (you!) push it across the line.

I probably won't have time for it within the next week or two.

Comment 6

[Nick Alexander :nalexander [he/him]]

Attachment: MozReview Request: Bug 1194365 - Bump Andoid NDK to r10e; manually install Android SDK and NDK on Mac OS X. r?mcomella

Bug 1194365 - Bump Andoid NDK to r10e; manually install Android SDK and NDK on Mac OS X. r?mcomella

This bumps the NDK version to r10e.

Previously, we used brew to install android-sdk and a custom version
of android-ndk.  That makes it hard to control the installed versions.
This installs from downloaded archives, which unifies the Mac OS X
approach with the straight-forward Linux approach.

Comment 7

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Builds on the implementation in bug 1171232.

Comment 8

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Comment on attachment 8664475 [details]
MozReview Request: Bug 1194365 - Bump Andoid NDK to r10e; manually install Android SDK and NDK on Mac OS X. r?mcomella

https://reviewboard.mozilla.org/r/19957/#review17999

I skimmed the code and it looks reasonable.

I tested locally pulling down the changesets and running mach bootstrap. I have no Android env var set so the android dependencies successfully downloaded to .mozbuild (in addition my homebrew install) and I was able to successfully build, compile, and run a Fennec instance.

Don't forget to update the gradle configs to look at .mozbuild for the Android SDK!

::: python/mozboot/mozboot/osx.py:338
(Diff revision 1)
> +            ('wget', 'wget'),

I'm guessing we should dupe the wget bug over to this one?

Comment 9

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/5ecda9616475

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/5ecda9616475