Skip to content

Commit ab3ca61

Browse files
authored
Merge pull request #6463 from BOINC/vko_fix_vcpkg_android_build
[ci][android] build vcpkg test on r23c and r27c NDKs
2 parents b196205 + 8d48769 commit ab3ca61

3 files changed

Lines changed: 16 additions & 10 deletions

File tree

.github/workflows/vcpkg_test.yml

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,7 @@ jobs:
5959
strategy:
6060
matrix:
6161
triplet: [ 'arm-android', 'arm-neon-android', 'arm64-android', 'x64-android', 'x86-android' ]
62+
ndk: [ 'r23c', 'r27c' ]
6263
fail-fast: false
6364
steps:
6465
- name: Checkout
@@ -89,17 +90,17 @@ jobs:
8990
9091
- name: Install Android NDK
9192
run: |
92-
curl -L -o android-ndk.zip https://dl.google.com/android/repository/android-ndk-r23c-linux.zip
93+
curl -L -o android-ndk.zip https://dl.google.com/android/repository/android-ndk-${{ matrix.ndk }}-linux.zip
9394
unzip android-ndk.zip
9495
9596
- name: Prepare build
9697
run: |
9798
mkdir -p tests/vcpkg/build
9899
99100
- name: Configure arm build
100-
if: ${{ matrix.triplet == 'arm-android' }}
101+
if: ${{ matrix.triplet == 'arm-android' && matrix.ndk == 'r23c' }}
101102
run: |
102-
export ANDROID_NDK_HOME=$(pwd)/android-ndk-r23c
103+
export ANDROID_NDK_HOME=$(pwd)/android-ndk-${{ matrix.ndk }}
103104
echo "ANDROID_NDK_HOME=${ANDROID_NDK_HOME}" >> $GITHUB_ENV
104105
toolchain_root="${ANDROID_NDK_HOME}/toolchains/llvm/prebuilt/linux-x86_64"
105106
tc_binaries="${toolchain_root}/bin"
@@ -119,7 +120,7 @@ jobs:
119120
- name: Configure arm-neon build
120121
if: ${{ matrix.triplet == 'arm-neon-android' }}
121122
run: |
122-
export ANDROID_NDK_HOME="$(pwd)/android-ndk-r23c"
123+
export ANDROID_NDK_HOME="$(pwd)/android-ndk-${{ matrix.ndk }}"
123124
echo "ANDROID_NDK_HOME=${ANDROID_NDK_HOME}" >> $GITHUB_ENV
124125
toolchain_root="${ANDROID_NDK_HOME}/toolchains/llvm/prebuilt/linux-x86_64"
125126
tc_binaries="${toolchain_root}/bin"
@@ -139,7 +140,7 @@ jobs:
139140
- name: Configure arm64 build
140141
if: ${{ matrix.triplet == 'arm64-android' }}
141142
run: |
142-
export ANDROID_NDK_HOME="$(pwd)/android-ndk-r23c"
143+
export ANDROID_NDK_HOME="$(pwd)/android-ndk-${{ matrix.ndk }}"
143144
echo "ANDROID_NDK_HOME=${ANDROID_NDK_HOME}" >> $GITHUB_ENV
144145
toolchain_root="${ANDROID_NDK_HOME}/toolchains/llvm/prebuilt/linux-x86_64"
145146
tc_binaries="${toolchain_root}/bin"
@@ -159,7 +160,7 @@ jobs:
159160
- name: Configure x64 build
160161
if: ${{ matrix.triplet == 'x64-android' }}
161162
run: |
162-
export ANDROID_NDK_HOME="$(pwd)/android-ndk-r23c"
163+
export ANDROID_NDK_HOME="$(pwd)/android-ndk-${{ matrix.ndk }}"
163164
echo "ANDROID_NDK_HOME=${ANDROID_NDK_HOME}" >> $GITHUB_ENV
164165
toolchain_root="${ANDROID_NDK_HOME}/toolchains/llvm/prebuilt/linux-x86_64"
165166
tc_binaries="${toolchain_root}/bin"
@@ -179,7 +180,7 @@ jobs:
179180
- name: Configure x86 build
180181
if: ${{ matrix.triplet == 'x86-android' }}
181182
run: |
182-
export ANDROID_NDK_HOME="$(pwd)/android-ndk-r23c"
183+
export ANDROID_NDK_HOME="$(pwd)/android-ndk-${{ matrix.ndk }}"
183184
echo "ANDROID_NDK_HOME=${ANDROID_NDK_HOME}" >> $GITHUB_ENV
184185
toolchain_root="${ANDROID_NDK_HOME}/toolchains/llvm/prebuilt/linux-x86_64"
185186
tc_binaries="${toolchain_root}/bin"
@@ -197,14 +198,15 @@ jobs:
197198
cmake -E env CC="${CC}" CXX="${CXX}" LD="${LD}" CFLAGS="${CFLAGS}" CXXFLAGS="${CXXFLAGS}" LDFLAGS="${LDFLAGS}" cmake -B tests/vcpkg/build -S tests/vcpkg/ -DVCPKG_TARGET_TRIPLET=${{ matrix.triplet }}
198199
199200
- name: Build
201+
if: ${{ matrix.triplet != 'arm-android' || (matrix.triplet == 'arm-android' && matrix.ndk == 'r23c') }}
200202
run: |
201203
cmake --build tests/vcpkg/build --config Release
202204
203205
- name: Upload logs on failure
204206
if: ${{ failure() }}
205207
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02
206208
with:
207-
name: vcpkg_test-logs-${{ matrix.triplet }}
209+
name: vcpkg_test-logs-${{ matrix.triplet }}-${{ matrix.ndk }}
208210
path: vcpkg/buildtrees/**/*.log
209211

210212
linux:

configure.ac

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -734,10 +734,10 @@ sse3_flags="-msse3"
734734
avx_flags="-mavx"
735735
CXXFLAGS="${save_cxxflags} ${sse3_flags}"
736736
CPPFLAGS="${save_cppflags} ${sse3_flags}"
737-
AC_LINK_IFELSE([AC_LANG_PROGRAM([],)], [], [sse_flags=""])
737+
AC_COMPILE_IFELSE([AC_LANG_PROGRAM([],)], [], [sse_flags=""])
738738
CXXFLAGS="${save_cxxflags} ${avx_flags}"
739739
CPPFLAGS="${save_cppflags} ${avx_flags}"
740-
AC_LINK_IFELSE([AC_LANG_PROGRAM([],)], [], [avx_flags=""])
740+
AC_COMPILE_IFELSE([AC_LANG_PROGRAM([],)], [], [avx_flags=""])
741741
CXXFLAGS="${save_cxxflags} ${sse3_flags}"
742742
CXXFLAGS="${save_cxxflags} ${sse3_flags}"
743743
AC_CHECK_HEADERS([intrin.h x86intrin.h pmmintrin.h xmmintrin.h emmintrin.h])

lib/diagnostics.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,10 @@
4242
#include "mac_backtrace.h"
4343
#endif
4444

45+
#if defined(ANDROID) && __ANDROID_API__ < 33
46+
#undef HAVE_EXECINFO_H
47+
#endif
48+
4549
#ifdef HAVE_EXECINFO_H
4650
#include <execinfo.h>
4751
#endif

0 commit comments

Comments
 (0)