From a2df75c5a3df2e4e3f5ffbb666de5f07c8c61890 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 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/macos.yml b/.github/workflows/macos.yml index 6c481170df..8b04c82eac 100644 --- a/.github/workflows/macos.yml +++ b/.github/workflows/macos.yml @@ -26,7 +26,7 @@ jobs: cmake --build build/Release - name: Test ninja - run: ctest -vv + run: ./ninja_test working-directory: build/Release - name: Create ninja archive