From 6e77cdf3a072f21bc5755f07a66402f8748e86d0 Mon Sep 17 00:00:00 2001 From: Michael Pratt Date: Sun, 18 Aug 2024 20:19:10 -0400 Subject: [PATCH] GitHub Actions: Remove usage of CTest It seems that new versions of CTest now require case-sensitive arguments, resulting in an "unknown argument" error for "-vv", while documentation shows the argument for the full verbosity option as "-VV". In CMakeLists.txt, there is currently only 1 test registered to CMake with add_test(), and that is to run the ninja_test binary. Since using CTest is not reducing the number of commands used for testing, fix the testing invocation by replacing CTest with a direct run of the test. Signed-off-by: Michael Pratt --- .github/workflows/macos.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/macos.yml b/.github/workflows/macos.yml index f4f26abf69..bec80b92d8 100644 --- a/.github/workflows/macos.yml +++ b/.github/workflows/macos.yml @@ -24,9 +24,9 @@ jobs: cmake -Bbuild -GXcode '-DCMAKE_OSX_ARCHITECTURES=arm64;x86_64' cmake --build build --config Release - - name: Test ninja - run: ctest -C Release -vv - working-directory: build + - name: Test ninja (Release) + run: ./ninja_test + working-directory: build/Release - name: Create ninja archive shell: bash