Closed Bug 1648552 Opened 4 years ago Closed 4 years ago

mach bootstrap does not check if Android NDK version in path is the correct version

Categories

(Firefox Build System :: Bootstrap Configuration, defect, P3)

Unspecified
All
defect

Tracking

(firefox80 fixed)

RESOLVED FIXED
mozilla80
Tracking Status
firefox80 --- fixed

People

(Reporter: jonalmeida, Assigned: rstewart)

Details

(Keywords: in-triage)

Attachments

(1 file)

When build GeckoView with a non-artifact setup, I had the Android NDK 20b version in my /usr/local/share/android-ndk.

I tried running mach build and ran into this:

 0:06.49 ERROR: The only supported version of the NDK is r20 (have r20b)
 0:06.49 Please run |mach bootstrap| to install the correct NDK.
 0:06.61 *** Fix above errors and then restart with               "./mach build"
 0:06.61 make: *** [configure] Error 1

After running mach bootstrap, it would tell me I already had the NDK and would not proceed to download the correct one:

Your version of Java (/Library/Java/JavaVirtualMachines/adoptopenjdk-8.jdk/Contents/Home/bin/java) is at least 1.8 (1.8).

Looks like you have the Android NDK installed at:
/usr/local/share/android-ndk


Looks like you have the Android SDK installed at:
/Users/jalmeida/.mozbuild/android-sdk-macosx
We will install all required Android packages.


We are now installing the following Android packages:
platform-tools
build-tools;29.0.3
platforms;android-29
emulator

My resolution was needing to remove the NDK directory with brew cask uninstall android-ndk.

Not sure what system information would be important:

  • Mac OS 10.15.4 (19E287)

This is probably in the wrong component, please move it where fit.

Component: General → Bootstrap Configuration
Product: GeckoView → Firefox Build System
Severity: -- → S3
Type: enhancement → defect
Keywords: in-triage
Priority: -- → P3
Assignee: nobody → rstewart
Pushed by rstewart@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/49cd46651c7b
`mach bootstrap` checks if Android NDK version is the correct version r=nalexander
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: