From d0e0aa92d74c8b9fccc24ca37030a01c683790d9 Mon Sep 17 00:00:00 2001
From: Chris B <cbieneman@microsoft.com>
Date: Mon, 28 Nov 2022 11:16:55 -0600
Subject: [PATCH] Update GitHub workflow to (hopefully) work (#4812)

There were a few things missing in the initial attempt at this workflow. I left off the file extension for the CMake cache file, and also forgot to initialize the submodules.

Additionally using make is... slow. Using make `-j` causes the VM to run out of memory. To resolve those issues I'm installing ninja and using ninja instead. I'm also setting `LLVM_USE_LINKER=lld` to reduce linker memory usage and `LLVM_PARALLEL_LINK_JOBS=1` to only run one link at a time.

I've also added `--rerun-failed` and `--output-on-failure` to the ctest invocation so that if it fails we get some hopefully helpful output.
---
 .github/workflows/coverage-gh-pages.yml | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/coverage-gh-pages.yml b/.github/workflows/coverage-gh-pages.yml
index 224175003..45eeccbd6 100644
--- a/.github/workflows/coverage-gh-pages.yml
+++ b/.github/workflows/coverage-gh-pages.yml
@@ -27,16 +27,20 @@ jobs:
     steps:
       - name: Checkout
         uses: actions/checkout@v3
+        with:
+          submodules: true
       - name: Setup Pages
         uses: actions/configure-pages@v2
+      - name: Install dependencies
+        run: sudo apt install -y ninja-build
       - name: Configure
-        run: mkdir build && cd build && cmake -DCMAKE_BUILD_TYPE=Release -DDXC_COVERAGE=On -C ${{github.workspace}}/cmake/caches/PredefinedParams -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ ${{github.workspace}}
+        run: mkdir build && cd build && cmake -G Ninja -DLLVM_PARALLEL_LINK_JOBS=1 -DLLVM_USE_LINKER=lld -DCMAKE_BUILD_TYPE=Release -DDXC_COVERAGE=On -C ${{github.workspace}}/cmake/caches/PredefinedParams.cmake -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ ${{github.workspace}}
       - name: Build
-        run: cd build && make
+        run: cd build && ninja
       - name: Test
-        run: cd build && ctest
+        run: cd build && ctest --rerun-failed --output-on-failure
       - name: Generate Report
-        run: cd build && make generate-coverage-report
+        run: cd build && ninja generate-coverage-report
       - name: Upload artifact
         uses: actions/upload-pages-artifact@v1
         with: