From 67e5a1bba8968266c2f6922379a6d39de1d01696 Mon Sep 17 00:00:00 2001
From: Christoph Knittel <ck@cca.io>
Date: Sun, 2 Nov 2025 09:19:08 +0100
Subject: [PATCH] Fix: do not rebuild ninja when building playground

---
 .github/workflows/ci.yml | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 197dbf49fc..86c228c57f 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -57,6 +57,7 @@ jobs:
             upload_binaries: true
             node-target: win32-x64
             rust-target: x86_64-pc-windows-gnu
+            exe-suffix: ".exe"
 
           # Verify that the compiler still builds with the oldest OCaml version we support.
           - os: ubuntu-24.04
@@ -137,9 +138,9 @@ jobs:
 
       - name: Copy rewatch binary
         run: |
-          cp rewatch/target/${{ matrix.rust-target }}/release/rescript${{ runner.os == 'Windows' && '.exe' || '' }} rescript
+          cp rewatch/target/${{ matrix.rust-target }}/release/rescript${{ matrix.exe-suffix }} rescript
           mkdir -p rewatch/target/release
-          cp rewatch/target/${{ matrix.rust-target }}/release/rescript${{ runner.os == 'Windows' && '.exe' || '' }} rewatch/target/release
+          cp rewatch/target/${{ matrix.rust-target }}/release/rescript${{ matrix.exe-suffix }} rewatch/target/release
           ./scripts/copyExes.js --rewatch
         shell: bash
 
@@ -294,7 +295,7 @@ jobs:
         id: ninja-build-cache
         uses: actions/cache@v4
         with:
-          path: ninja/ninja${{ runner.os == 'Windows' && '.exe' || '' }}
+          path: ninja/ninja${{ matrix.exe-suffix }}
           key: ninja-build-v2-${{ matrix.os }}-${{ hashFiles('ninja/src/**') }}
 
       - name: Setup Python for ninja build
@@ -319,6 +320,11 @@ jobs:
           LDFLAGS: -static
         run: node scripts/buildNinjaBinary.js
 
+      # Prevent ninja being rebuilt on playground build step later
+      - name: Touch ninja/ninja
+        if: ${{ matrix.build_playground && steps.ninja-build-cache.outputs.cache-hit == 'true' }}
+        run: touch ninja/ninja${{ matrix.exe-suffix }}
+
       - name: Copy ninja exe to platform bin dir
         run: node scripts/copyExes.js --ninja