From e20f0e3acd47d30a61dd66351f0feb994c571320 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Storsj=C3=B6?= Date: Mon, 12 May 2025 10:49:42 +0300 Subject: [PATCH] github: Stop manually installing ninja Since https://github.com/actions/runner-images/pull/11706, https://github.com/actions/runner-images/pull/11707 and https://github.com/actions/runner-images/pull/11737, ninja is preinstalled in the Ubuntu, macOS and x86 Windows runner images. (See https://github.com/actions/runner-images/issues/11391 for the overall issue where this was finally agreed to be installed.) It is still missing in the windows-11-arm images though, so keep installing it in any jobs running on Windows. --- .github/workflows/build.yml | 7 ++----- .github/workflows/msvcrt.yml | 3 +-- 2 files changed, 3 insertions(+), 7 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 6391624..df1ee49 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -79,7 +79,6 @@ jobs: SOURCE_DATE_EPOCH: ${{needs.prepare.outputs.COMMIT_DATE_UNIX}} BUILD_DATE: ${{needs.prepare.outputs.BUILD_DATE}} run: | - sudo apt-get update && sudo apt-get install ninja-build # Skip dynamic library dependencies that might make it harder to # run the binaries on other distros (and that have little use within # llvm-mingw). @@ -113,7 +112,7 @@ jobs: MINGW_W64_VERSION: ${{needs.prepare.outputs.MINGW_W64_VERSION}} SOURCE_DATE_EPOCH: ${{needs.prepare.outputs.COMMIT_DATE_UNIX}} run: | - sudo apt-get update && sudo apt-get install ninja-build g++-aarch64-linux-gnu + sudo apt-get update && sudo apt-get install g++-aarch64-linux-gnu ./build-all.sh $(pwd)/install/llvm-mingw --no-runtimes --host=aarch64-linux-gnu .github/workflows/store-version.sh install/llvm-mingw/versions.txt - uses: actions/download-artifact@v4 @@ -169,7 +168,6 @@ jobs: SOURCE_DATE_EPOCH: ${{needs.prepare.outputs.COMMIT_DATE_UNIX}} BUILD_DATE: ${{needs.prepare.outputs.BUILD_DATE}} run: | - sudo apt-get update && sudo apt-get install ninja-build # Skip dynamic library dependencies that might make it harder to # run the binaries on other distros (and that have little use within # llvm-mingw). @@ -200,7 +198,6 @@ jobs: SOURCE_DATE_EPOCH: ${{needs.prepare.outputs.COMMIT_DATE_UNIX}} BUILD_DATE: ${{needs.prepare.outputs.BUILD_DATE}} run: | - brew install ninja # Disable zstd and python. Both are available on the runners, but # installed with homebrew, and only available in the native (x86_64) # form. Therefore, autodetection will pick them up, but linking @@ -313,7 +310,7 @@ jobs: SOURCE_DATE_EPOCH: ${{needs.prepare.outputs.COMMIT_DATE_UNIX}} BUILD_DATE: ${{needs.prepare.outputs.BUILD_DATE}} run: | - sudo apt-get update && sudo apt-get install libltdl-dev swig autoconf-archive ninja-build + sudo apt-get update && sudo apt-get install libltdl-dev swig autoconf-archive ./build-cross-tools.sh /opt/llvm-mingw $(pwd)/install/llvm-mingw ${{matrix.arch}} --with-python .github/workflows/store-version.sh install/llvm-mingw/versions.txt cd install diff --git a/.github/workflows/msvcrt.yml b/.github/workflows/msvcrt.yml index c163ef5..1a9a54a 100644 --- a/.github/workflows/msvcrt.yml +++ b/.github/workflows/msvcrt.yml @@ -75,7 +75,6 @@ jobs: SOURCE_DATE_EPOCH: ${{needs.prepare.outputs.COMMIT_DATE_UNIX}} BUILD_DATE: ${{needs.prepare.outputs.BUILD_DATE}} run: | - sudo apt-get update && sudo apt-get install ninja-build ./build-all.sh $(pwd)/install/llvm-mingw --no-tools --wipe-runtimes --with-default-msvcrt=msvcrt .github/workflows/store-version.sh install/llvm-mingw/versions.txt cd install @@ -119,7 +118,7 @@ jobs: PYTHON_VERSION_MINGW: ${{needs.prepare.outputs.PYTHON_VERSION_MINGW}} TAG: ${{needs.prepare.outputs.TAG}} run: | - sudo apt-get update && sudo apt-get install libltdl-dev swig autoconf-archive ninja-build + sudo apt-get update && sudo apt-get install libltdl-dev swig autoconf-archive ./build-cross-tools.sh /opt/llvm-mingw $(pwd)/install/llvm-mingw ${{matrix.arch}} --with-python .github/workflows/store-version.sh install/llvm-mingw/versions.txt cd install