Skip to content

Commit

Permalink
Update Android NDK (r25) to match what is used in pre-built Qt libraries
Browse files Browse the repository at this point in the history
  • Loading branch information
nirvn committed Dec 10, 2023
1 parent 03e8b27 commit 102713d
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 8 deletions.
2 changes: 1 addition & 1 deletion .docker/android_dev/Dockerfile
@@ -1,5 +1,5 @@
FROM ubuntu:22.04
ARG NDK_VERSION=23.1.7779620
ARG NDK_VERSION=25.1.8937393

ENV ANDROID_SDK_HOME /opt/android-sdk-linux
ENV ANDROID_SDK_ROOT /opt/android-sdk-linux
Expand Down
11 changes: 7 additions & 4 deletions .github/workflows/android.yml
Expand Up @@ -52,9 +52,9 @@ jobs:
id: vars
shell: bash
run: |
echo "ANDROID_NDK_VERSION=23.1.7779620" >> $GITHUB_ENV
echo "ANDROID_BUILD_TOOLS_VERSION=30.0.3" >> $GITHUB_ENV
echo "ANDROID_NDK_HOME=/usr/local/lib/android/sdk/ndk/23.1.7779620" >> $GITHUB_ENV
echo "ANDROID_NDK_VERSION=25.1.8937393" >> $GITHUB_ENV
echo "ANDROID_BUILD_TOOLS_VERSION=33.0.0" >> $GITHUB_ENV
echo "ANDROID_NDK_HOME=/usr/local/lib/android/sdk/ndk/25.1.8937393" >> $GITHUB_ENV
echo "ndk.dir=$ANDROID_NDK_HOME" >> local.properties
ALL_FILES_ACCESS=${{ matrix.all_files_access }} ./scripts/ci/env_gh.sh
Expand Down Expand Up @@ -121,6 +121,9 @@ jobs:
TRIPLET=${{ matrix.triplet }} ALL_FILES_ACCESS=${{ matrix.all_files_access }} source ./scripts/version_number.sh
TRIPLET=${{ matrix.triplet }} ALL_FILES_ACCESS=${{ matrix.all_files_access }} source ./scripts/ci/generate-version-details.sh
sudo apt remove libgles2
sudo apt remove libegl1
cmake -S "${{ github.workspace }}" \
-B "${CMAKE_BUILD_DIR}" \
-G Ninja \
Expand All @@ -136,7 +139,7 @@ jobs:
-D WITH_SPIX=OFF \
-D CMAKE_PREFIX_PATH=${Qt6_DIR} \
-D QT_HOST_PATH=/home/runner/work/QField/Qt/${{ env.INSTALL_QT_VERSION }}/gcc_64 \
-D QT_HOST_PATH_CMAKE_DIR:PATH=/home/runner/work/QField/Qt/${{ env.INSTALL_QT_VERSION }}/gcc_64 \
-D QT_HOST_PATH_CMAKE_DIR:PATH=/home/runner/work/QField/Qt/${{ env.INSTALL_QT_VERSION }}/gcc_64/lib/cmake \
-D APP_VERSION="${APP_VERSION}" \
-D APK_VERSION_CODE="${APK_VERSION_CODE}" \
-D APP_VERSION_STR="${APP_VERSION_STR}" \
Expand Down
5 changes: 3 additions & 2 deletions cmake/Platform.cmake
Expand Up @@ -20,8 +20,9 @@ elseif(VCPKG_TARGET_TRIPLET STREQUAL "x86-android")
endif()

if(ANDROID_ABI)
set(ANDROID_NDK_VERSION "$ENV{ANDROID_NDK_HOME}" CACHE STRING "Android NDK version")
set(ANDROID_BUILD_TOOLS_VERSION "30.0.3" CACHE STRING "Android build-tools version")
set(ANDROID_NDK_VERSION "$ENV{ANDROID_NDK_VERSION}" CACHE STRING "Android NDK version")
set(ANDROID_NDK_HOME "$ENV{ANDROID_NDK_HOME}" CACHE STRING "Android NDK home path")
set(ANDROID_BUILD_TOOLS_VERSION "33.0.0" CACHE STRING "Android build-tools version")
set(ANDROID_TARGET_PLATFORM 33 CACHE INT "Target Android platform SDK version")
set(ANDROID_PLATFORM 21 CACHE INT "Minimum Android platform SDK version")
set(ANDROID_PLATFORM_INT 21 CACHE INT "Minimum Android platform SDK version") # Used in build.gradle.in
Expand Down
1 change: 0 additions & 1 deletion scripts/build-vcpkg.sh
Expand Up @@ -18,7 +18,6 @@ echo "Package name ${APP_PACKAGE_NAME}"
cmake -S "${SOURCE_DIR}" \
-B "${CMAKE_BUILD_DIR}" \
-G Ninja \
-D BUILD_WITH_QT6=ON \
-D CMAKE_PREFIX_PATH=${Qt6_DIR} \
-D QT_HOST_PATH=/home/devel/${install_qt_version}/gcc_64 \
-D QT_HOST_PATH_CMAKE_DIR=/home/devel/${install_qt_version}/gcc_64/lib/cmake \
Expand Down

1 comment on commit 102713d

@qfield-fairy
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.