Skip to content

Commit 9f6c54d

Browse files
marc-hbfabiobaltieri
authored andcommitted
action: do not re-install ninja
After many years asking, Github runners finally got ninja by default in April 2025: - actions/runner-images#11235 So, stop re-installing ninja - at least not unless there is a specific and documented reason to do so. This saves a little bit of time and fixes the following warning in hello_world_multiplatform.yaml: https://github.com/zephyrproject-rtos/zephyr/actions/runs/19853412716 ``` build (macos-14) ninja 1.13.2 is already installed and up-to-date. To reinstall 1.13.2, run: brew reinstall ninja ``` Do not change the Windows build yet because the build environment is always much more complicated on Windows, so let's first make sure this is fine on Linux and macOS. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
1 parent 8354324 commit 9f6c54d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

action.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -96,12 +96,12 @@ runs:
9696
if [ "${{ runner.os }}" = "Linux" ]; then
9797
sudo rm -f /var/lib/man-db/auto-update
9898
sudo apt-get update -y
99-
sudo apt-get install -y ninja-build ccache gperf
99+
sudo apt-get install -y ccache gperf
100100
if [ "${{ runner.arch }}" = "X64" ]; then
101101
sudo apt-get install -y libc6-dev-i386 g++-multilib
102102
fi
103103
elif [ "${{ runner.os }}" = "macOS" ]; then
104-
brew install ninja ccache qemu dtc gperf
104+
brew install ccache qemu dtc gperf
105105
elif [ "${{ runner.os }}" = "Windows" ]; then
106106
choco feature enable -n allowGlobalConfirmation
107107
choco install ninja wget gperf

0 commit comments

Comments
 (0)