Skip to content

Commit fcde5d1

Browse files
author
MacroFake
committed
Merge bitcoin/bitcoin#25255: ci: Improve "ARM64 Android APK" task
c47944f ci: Reuse some configure options in "ARM64 Android APK" task (Hennadii Stepanov) 7739438 ci, android: Update NDK up to r23c (Hennadii Stepanov) ca0c3e5 ci, android: Update Command-line Tools from 2.1 up to 7.0 (Hennadii Stepanov) 8790da3 ci: Drop unneeded packages in "ARM64 Android APK" task (Hennadii Stepanov) Pull request description: This PR improves the "ARM64 Android APK" CI task in the following ways: - dropped packages that are not required to be installed - updated Android Command-line Tools and Android NDK to make the CI environment closer to the default one, which is provided by Android Studio ACKs for top commit: icota: utACK c47944f Tree-SHA512: 45f5aba41007a502ae90333272370fd559c48a27d573896c449b3e436c5cf2b6440408381e4d20eb53104426ade26d3a9014c09dcdf3257ec897a537095efa4f
2 parents 1b2e1d1 + c47944f commit fcde5d1

File tree

4 files changed

+14
-12
lines changed

4 files changed

+14
-12
lines changed

.cirrus.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -324,7 +324,7 @@ task:
324324
<< : *BASE_TEMPLATE
325325
android_sdk_cache:
326326
folder: "depends/SDKs/android"
327-
fingerprint_key: "ANDROID_API_LEVEL=28 ANDROID_BUILD_TOOLS_VERSION=28.0.3 ANDROID_NDK_VERSION=23.1.7779620"
327+
fingerprint_key: "ANDROID_API_LEVEL=28 ANDROID_BUILD_TOOLS_VERSION=28.0.3 ANDROID_NDK_VERSION=23.2.8568313"
328328
depends_sources_cache:
329329
folder: "depends/sources"
330330
fingerprint_script: git rev-list -1 HEAD ./depends

ci/test/00_setup_env_android.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
export LC_ALL=C.UTF-8
88

99
export HOST=aarch64-linux-android
10-
export PACKAGES="clang llvm unzip openjdk-8-jdk gradle"
10+
export PACKAGES="unzip openjdk-8-jdk gradle"
1111
export CONTAINER_NAME=ci_android
1212
export DOCKER_NAME_TAG="ubuntu:focal"
1313

@@ -16,8 +16,8 @@ export RUN_FUNCTIONAL_TESTS=false
1616

1717
export ANDROID_API_LEVEL=28
1818
export ANDROID_BUILD_TOOLS_VERSION=28.0.3
19-
export ANDROID_NDK_VERSION=23.1.7779620
20-
export ANDROID_TOOLS_URL=https://dl.google.com/android/repository/commandlinetools-linux-6609375_latest.zip
19+
export ANDROID_NDK_VERSION=23.2.8568313
20+
export ANDROID_TOOLS_URL=https://dl.google.com/android/repository/commandlinetools-linux-8512546_latest.zip
2121
export ANDROID_HOME="${DEPENDS_DIR}/SDKs/android"
2222
export ANDROID_NDK_HOME="${ANDROID_HOME}/ndk/${ANDROID_NDK_VERSION}"
2323
export DEP_OPTS="ANDROID_SDK=${ANDROID_HOME} ANDROID_NDK=${ANDROID_NDK_HOME} ANDROID_API_LEVEL=${ANDROID_API_LEVEL} ANDROID_TOOLCHAIN_BIN=${ANDROID_NDK_HOME}/toolchains/llvm/prebuilt/linux-x86_64/bin/"

ci/test/05_before_script.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,9 +31,9 @@ if [ -n "$ANDROID_HOME" ] && [ ! -d "$ANDROID_HOME" ]; then
3131
if [ ! -f "$ANDROID_TOOLS_PATH" ]; then
3232
CI_EXEC curl --location --fail "${ANDROID_TOOLS_URL}" -o "$ANDROID_TOOLS_PATH"
3333
fi
34-
CI_EXEC mkdir -p "${ANDROID_HOME}/cmdline-tools"
35-
CI_EXEC unzip -o "$ANDROID_TOOLS_PATH" -d "${ANDROID_HOME}/cmdline-tools"
36-
CI_EXEC "yes | ${ANDROID_HOME}/cmdline-tools/tools/bin/sdkmanager --install \"build-tools;${ANDROID_BUILD_TOOLS_VERSION}\" \"platform-tools\" \"platforms;android-${ANDROID_API_LEVEL}\" \"ndk;${ANDROID_NDK_VERSION}\""
34+
CI_EXEC mkdir -p "$ANDROID_HOME"
35+
CI_EXEC unzip -o "$ANDROID_TOOLS_PATH" -d "$ANDROID_HOME"
36+
CI_EXEC "yes | ${ANDROID_HOME}/cmdline-tools/bin/sdkmanager --sdk_root=\"${ANDROID_HOME}\" --install \"build-tools;${ANDROID_BUILD_TOOLS_VERSION}\" \"platform-tools\" \"platforms;android-${ANDROID_API_LEVEL}\" \"ndk;${ANDROID_NDK_VERSION}\""
3737
fi
3838

3939
if [ -z "$NO_DEPENDS" ]; then

ci/test/06_script_a.sh

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,18 +6,20 @@
66

77
export LC_ALL=C.UTF-8
88

9+
BITCOIN_CONFIG_ALL="--enable-suppress-external-warnings --disable-dependency-tracking --prefix=$DEPENDS_DIR/$HOST"
10+
if [ -z "$NO_WERROR" ]; then
11+
BITCOIN_CONFIG_ALL="${BITCOIN_CONFIG_ALL} --enable-werror"
12+
fi
13+
914
if [ -n "$ANDROID_TOOLS_URL" ]; then
1015
CI_EXEC make distclean || true
1116
CI_EXEC ./autogen.sh
12-
CI_EXEC ./configure "$BITCOIN_CONFIG" --prefix="${DEPENDS_DIR}/aarch64-linux-android" || ( (CI_EXEC cat config.log) && false)
17+
CI_EXEC ./configure "$BITCOIN_CONFIG_ALL" "$BITCOIN_CONFIG" || ( (CI_EXEC cat config.log) && false)
1318
CI_EXEC "make $MAKEJOBS && cd src/qt && ANDROID_HOME=${ANDROID_HOME} ANDROID_NDK_HOME=${ANDROID_NDK_HOME} make apk"
1419
exit 0
1520
fi
1621

17-
BITCOIN_CONFIG_ALL="--enable-external-signer --enable-suppress-external-warnings --disable-dependency-tracking --prefix=$DEPENDS_DIR/$HOST --bindir=$BASE_OUTDIR/bin --libdir=$BASE_OUTDIR/lib"
18-
if [ -z "$NO_WERROR" ]; then
19-
BITCOIN_CONFIG_ALL="${BITCOIN_CONFIG_ALL} --enable-werror"
20-
fi
22+
BITCOIN_CONFIG_ALL="${BITCOIN_CONFIG_ALL} --enable-external-signer --bindir=$BASE_OUTDIR/bin --libdir=$BASE_OUTDIR/lib"
2123
CI_EXEC "ccache --zero-stats --max-size=$CCACHE_SIZE"
2224

2325
if [ -n "$CONFIG_SHELL" ]; then

0 commit comments

Comments
 (0)