From ddde0f1eeaac40ae879f1727813fcdc8045cb231 Mon Sep 17 00:00:00 2001
From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Fri, 10 Dec 2021 18:00:38 +0100
Subject: [PATCH] Reenable cmake install step for Windows builds

Just skip using sudo under this OS, but still run "cmake install" as
this is required for the final test, checking the installation success,
to work.
---
 .github/workflows/ci_cmake.yml | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/ci_cmake.yml b/.github/workflows/ci_cmake.yml
index 4c9df22855..2c9ad34727 100644
--- a/.github/workflows/ci_cmake.yml
+++ b/.github/workflows/ci_cmake.yml
@@ -65,6 +65,7 @@ jobs:
             cmake_tests: OFF
           - name: Windows MSVC
             runner: windows-latest
+            no_sudo: 1
             cmake_defines: -DCMAKE_C_COMPILER=cl.exe -DCMAKE_CXX_COMPILER=cl.exe
             cmake_generator: Ninja
             cmake_samples: SOME
@@ -129,10 +130,13 @@ jobs:
           cmake --build . -- $wxBUILD_ARGS
 
       - name: Installing
-        if: runner.os != 'Windows'
         working-directory: build_cmake
         run: |
-          sudo cmake --build . --target install
+            if [ -z "${{ matrix.no_sudo }}" ]; then
+                sudo cmake --build . --target install
+            else
+                cmake --build . --target install
+            fi
 
       - name: Testing
         if: matrix.cmake_tests != 'OFF'