]> git.ipfire.org Git - thirdparty/curl.git/commitdiff
GHA: always use Ninja on macos runners
authorViktor Szakats <commit@vsz.me>
Mon, 21 Apr 2025 02:00:36 +0000 (04:00 +0200)
committerViktor Szakats <commit@vsz.me>
Tue, 22 Apr 2025 14:50:12 +0000 (16:50 +0200)
It's installed by default on the macos runners now.

Closes #17115

.github/workflows/distcheck.yml
.github/workflows/macos.yml
.github/workflows/non-native.yml
.github/workflows/windows.yml

index 1347c86c64841a2a541c5cecf6e9e810643a63c5..1d6b7610505e1c41caea679d951b29c8d28168f1 100644 (file)
@@ -180,7 +180,7 @@ jobs:
           if [[ '${{ matrix.image }}' = *'ubuntu'* ]]; then
             sudo apt-get -o Dpkg::Use-Pty=0 install ninja-build libpsl-dev libssl-dev
           else
-            brew install ninja libpsl openssl
+            brew install libpsl openssl
           fi
 
       - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
index 35ae822e8f288121af9591067609c34679f7e5f8..2131feb2220e7244bc467afbc8306d55ba4e3adc 100644 (file)
@@ -461,7 +461,7 @@ jobs:
             [ '${{ matrix.config }}' = 'SecureTransport' ] && options+=' -DCURL_USE_SECTRANSP=ON'
             [ -n '${{ matrix.macos-version-min }}' ] && options+=' -DCMAKE_OSX_DEPLOYMENT_TARGET=${{ matrix.macos-version-min }}'
             # would pick up nghttp2, libidn2, and libssh2
-            cmake -B bld -D_CURL_PREFILL=ON \
+            cmake -B bld -G Ninja -D_CURL_PREFILL=ON \
               -DCMAKE_UNITY_BUILD=ON -DCURL_TEST_BUNDLES=ON -DCURL_WERROR=ON \
               -DCMAKE_OSX_SYSROOT="${sysroot}" \
               -DCMAKE_C_COMPILER_TARGET="$(uname -m | sed 's/arm64/aarch64/')-apple-darwin$(uname -r)" \
@@ -501,7 +501,12 @@ jobs:
           grep -F '#define' bld/lib/curl_config.h | sort || true
 
       - name: 'build / ${{ matrix.build }}'
-        run: make -C bld V=1 VERBOSE=1
+        run: |
+          if [ '${{ matrix.build }}' = 'cmake' ]; then
+            cmake --build bld --verbose
+          else
+            make -C bld V=1
+          fi
 
       - name: 'curl version'
         run: bld/src/curl --disable --version
index d92800655e8736f26072c5893d5b95614a0c9930..fac3cce28241f23b5439a7a547d37546c8f93285 100644 (file)
@@ -319,7 +319,7 @@ jobs:
             https://github.com/libressl/portable/releases/download/v${{ env.libressl-version }}/libressl-${{ env.libressl-version }}.tar.gz | tar -x
           cd libressl-${{ env.libressl-version }}
           # FIXME: on the 4.0.1 release, delete '-DHAVE_ENDIAN_H=0'
-          cmake -B . \
+          cmake -B . -G Ninja \
             -DHAVE_ENDIAN_H=0 \
             -DCMAKE_INSTALL_PREFIX="$HOME/libressl" \
             -DCMAKE_SYSTEM_NAME=iOS \
@@ -343,7 +343,7 @@ jobs:
           if [ -n '${{ matrix.build.generate }}' ]; then
             # https://cmake.org/cmake/help/latest/manual/cmake-toolchains.7.html#cross-compiling-for-ios-tvos-visionos-or-watchos
             [ -n '${{ matrix.build.generator }}' ] && options='-G ${{ matrix.build.generator }}'
-            cmake -B bld -D_CURL_PREFILL=ON \
+            cmake -B bld -G Ninja -D_CURL_PREFILL=ON \
               -DCMAKE_UNITY_BUILD=ON -DCURL_TEST_BUNDLES=ON -DCURL_WERROR=ON \
               -DCMAKE_SYSTEM_NAME=iOS \
               -DUSE_APPLE_IDN=ON \
index 53f33fca4b0f443edaf31e6f821546e7a085e8e6..58c85025093adb7b28e021bc1c81a435377f5163 100644 (file)
@@ -677,7 +677,7 @@ jobs:
         run: |
           PATH="$HOME/opt/mingw32ce/bin:$PATH"
           if [ '${{ matrix.build }}' = 'cmake' ]; then
-            cmake -B bld \
+            cmake -B bld -G Ninja \
               -DCMAKE_SYSTEM_NAME=WindowsCE \
               -DCMAKE_SYSTEM_VERSION=8.0 \
               -DCMAKE_SYSTEM_PROCESSOR=arm \