Skip to content

Commit

Permalink
GitHub Actions: Remove usage of CTest
Browse files Browse the repository at this point in the history
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 CTest invocation by replacing it with a direct run of the test.

Signed-off-by: Michael Pratt <[email protected]>
  • Loading branch information
mcprat committed Aug 24, 2024
1 parent dcefb83 commit c0f1521
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/macos.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ jobs:
cmake --build build --config Release
- name: Test ninja
run: ctest -C Release -vv
run: ./ninja_test
working-directory: build

- name: Create ninja archive
Expand Down

0 comments on commit c0f1521

Please sign in to comment.