[Openvpn-devel,XS] Change in openvpn[master]: Build on all four possible Android ABIs

Message ID 05494024a8ead03cb61ba2e1b772dcb40ff8a9e8-HTML@gerrit.openvpn.net
State New
Headers show
Series [Openvpn-devel,XS] Change in openvpn[master]: Build on all four possible Android ABIs | expand

Commit Message

stipa (Code Review) Feb. 13, 2025, 4:12 p.m. UTC
Attention is currently required from: flichtenheld.

Hello flichtenheld,

I'd like you to do a code review.
Please visit

    http://gerrit.openvpn.net/c/openvpn/+/897?usp=email

to review the following change.


Change subject: Build on all four possible Android ABIs
......................................................................

Build on all four possible Android ABIs

This also includes the last of the still used 32 bit platforms.
But the installed device base is getting thin as well.

Change-Id: I09d7e8cf4f15ae5810e0adafda15e489a3375892
---
M .github/workflows/build.yaml
1 file changed, 7 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.openvpn.net:29418/openvpn refs/changes/97/897/1

Patch

diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml
index 9022e23..4fe6532 100644
--- a/.github/workflows/build.yaml
+++ b/.github/workflows/build.yaml
@@ -39,10 +39,16 @@ 
     strategy:
       fail-fast: false
       matrix:
-        abi: [ arm64-v8a ]
+        abi: [ armeabi-v7a, arm64-v8a, x86_64, x86 ]
         include:
+          - abi: armeabi-v7a
+            vcpkg_triplet: arm-neon-android
           - abi: arm64-v8a
             vcpkg_triplet: arm64-android
+          - abi: x86_64
+            vcpkg_triplet: x64-android
+          - abi: x86
+            vcpkg_triplet: x86-android
     runs-on: ubuntu-24.04
     name: "Android - ${{ matrix.abi }}"
     # Github images already setup NDK with ANDROID_NDK_ROOT pointing to the root