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 testing invocation by replacing CTest with a direct run of the test.

Signed-off-by: Michael Pratt <[email protected]>
  • Loading branch information
mcprat committed Aug 26, 2024
1 parent dcefb83 commit 6e77cdf
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/macos.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 6e77cdf

Please sign in to comment.