From d160a87f30a8c1c752d8323f7b42f8ad46a6b450 Mon Sep 17 00:00:00 2001 From: Ryan VanderMeulen Date: Thu, 24 Oct 2024 14:21:14 -0400 Subject: [PATCH] Update NDK to r27c --- CHANGELOG.md | 3 +++ build.gradle | 2 +- components/support/rc_crypto/nss/nss_build_common/src/lib.rs | 2 +- docs/building.md | 4 ++-- docs/howtos/locally-building-jna.md | 2 +- taskcluster/docker/linux/Dockerfile | 2 +- 6 files changed, 9 insertions(+), 6 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 56233f68d..11ab06f99 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -8,6 +8,9 @@ ## 🦊 What's Changed 🦊 +### Android +- Upgraded NDK from r27 to r27c. ([#6432](https://github.com/mozilla/application-services/pull/6432)) + ### Glean - Updated to v61.2.0 ([#6410](https://github.com/mozilla/application-services/pull/6410)) diff --git a/build.gradle b/build.gradle index 960150ffb..28475e332 100644 --- a/build.gradle +++ b/build.gradle @@ -2,7 +2,7 @@ buildscript { ext.build = [ - ndkVersion: "27.0.12077973", // Keep it in sync in TC Dockerfile. + ndkVersion: "27.2.12479018", // Keep it in sync in TC Dockerfile. // In general, we should aim to keep these in sync with AC compileSdkVersion: 34, diff --git a/components/support/rc_crypto/nss/nss_build_common/src/lib.rs b/components/support/rc_crypto/nss/nss_build_common/src/lib.rs index c5fbf7f5c..3ac5a0780 100644 --- a/components/support/rc_crypto/nss/nss_build_common/src/lib.rs +++ b/components/support/rc_crypto/nss/nss_build_common/src/lib.rs @@ -92,7 +92,7 @@ fn link_nss_libs(kind: LinkingKind) { let target_arch = env::var("CARGO_CFG_TARGET_ARCH").unwrap(); if target_arch == "x86_64" && target_os == "android" { let android_home = env::var("ANDROID_HOME").expect("ANDROID_HOME not set"); - const ANDROID_NDK_VERSION: &str = "27.0.12077973"; + const ANDROID_NDK_VERSION: &str = "27.2.12479018"; // One of these will exist, depending on the host platform. const DARWIN_X86_64_LIB_DIR: &str = "/toolchains/llvm/prebuilt/darwin-x86_64/lib/clang/18/lib/linux/"; diff --git a/docs/building.md b/docs/building.md index fa4d1032c..e1c79e388 100644 --- a/docs/building.md +++ b/docs/building.md @@ -107,8 +107,8 @@ The following instructions assume that you are building `application-services` f 1. Set `ANDROID_SDK_ROOT` and `ANDROID_HOME` to the Android Studio sdk location and add it to your rc file (either `.zshrc` or `.bashrc` depending on the shell you use for your terminal). 1. Configure the required versions of NDK `Configure menu > System Settings > Android SDK > SDK Tools > NDK > Show Package Details > NDK (Side by side)` - - 26.2.11394342 (required by Fenix; note: a specific NDK version isn't configured, this maps to default [NDK version](https://developer.android.com/studio/projects/install-ndk#default-ndk-per-agp) for the [AGP version](https://searchfox.org/mozilla-central/source/mobile/android/android-components/plugins/dependencies/src/main/java/DependenciesPlugin.kt#33)) - - 27.0.12077973 (required by Application Services, [as configured](https://github.com/mozilla/application-services/blob/main/build.gradle#L33)) + - 27.0.12077973 (required by Fenix; note: a specific NDK version isn't configured, this maps to default [NDK version](https://developer.android.com/studio/projects/install-ndk#default-ndk-per-agp) for the [AGP version](https://searchfox.org/mozilla-central/source/mobile/android/android-components/plugins/dependencies/src/main/java/DependenciesPlugin.kt#33)) + - 27.2.12479018 (required by Application Services, [as configured](https://github.com/mozilla/application-services/blob/main/build.gradle#L33)) 1. If you are on Windows using WSL - drop to the section below, [Windows setup for Android (WSL)](building.md#windows-setup-for-android-via-wsl) before proceeding. 1. Check dependencies, environment variables diff --git a/docs/howtos/locally-building-jna.md b/docs/howtos/locally-building-jna.md index c7e9c54cc..b24b45877 100644 --- a/docs/howtos/locally-building-jna.md +++ b/docs/howtos/locally-building-jna.md @@ -17,7 +17,7 @@ Here are the steps that worked for me: for each target platform are in your `$PATH`. On my Mac with Android Studio the config was as follows: ``` - export NDK_ROOT="$HOME/Library/Android/sdk/ndk/27.0.12077973" + export NDK_ROOT="$HOME/Library/Android/sdk/ndk/27.2.12479018" export NDK_PLATFORM="$NDK_ROOT/platforms/android-25" export PATH="$PATH:$NDK_ROOT/toolchains/llvm/prebuilt/darwin-x86_64/bin" export PATH="$PATH:$NDK_ROOT/toolchains/aarch64-linux-android-4.9/prebuilt/darwin-x86_64/bin" diff --git a/taskcluster/docker/linux/Dockerfile b/taskcluster/docker/linux/Dockerfile index 05d35cb33..d1826d565 100644 --- a/taskcluster/docker/linux/Dockerfile +++ b/taskcluster/docker/linux/Dockerfile @@ -22,7 +22,7 @@ WORKDIR /builds/worker/ ENV ANDROID_BUILD_TOOLS "35.0.0" ENV ANDROID_TOOLS_VERSION "11076708" ENV ANDROID_PLATFORM_VERSION "35" -ENV ANDROID_NDK_VERSION "27.0.12077973" +ENV ANDROID_NDK_VERSION "27.2.12479018" # Set up the language variables to avoid problems (we run locale-gen later). ENV LANG en_US.UTF-8