Skip to content

Commit

Permalink
CI: cleanup and fix linux valgrind tests
Browse files Browse the repository at this point in the history
Signed-off-by: Martin Olivier <[email protected]>
  • Loading branch information
martin-olivier committed Jan 9, 2025
1 parent 1c19f43 commit 8bc1eb1
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 6 deletions.
17 changes: 11 additions & 6 deletions .github/workflows/CI.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,9 @@ defaults:
run:
shell: bash


jobs:
unit_tests_windows:
windows_msvc:
strategy:
fail-fast: false
matrix:
Expand All @@ -44,7 +45,8 @@ jobs:
working-directory: build
run: ctest --verbose

unit_tests_msys2:

windows_msys2:
defaults:
run:
shell: msys2 {0}
Expand Down Expand Up @@ -76,7 +78,8 @@ jobs:
working-directory: build
run: ctest --verbose

unit_tests_linux:

linux:
strategy:
fail-fast: false
matrix:
Expand All @@ -90,9 +93,10 @@ jobs:
docker_arch: i386

name: linux ${{ matrix.arch }} ${{ matrix.action }} / C++${{ matrix.cxx-std }} / gcc)
runs-on: ubuntu-24.04
runs-on: ubuntu-latest
container:
image: ${{ matrix.docker_arch }}/debian:latest
# replace with debian 13 (or latest) when available
image: ${{ matrix.docker_arch }}/debian:testing
options: --user root
volumes:
- ${{ github.workspace }}:/workspace
Expand Down Expand Up @@ -123,7 +127,8 @@ jobs:
working-directory: build
run: valgrind --leak-check=full --show-leak-kinds=all --error-exitcode=1 ./unit_tests

unit_tests_macos:

macos:
strategy:
fail-fast: false
matrix:
Expand Down
6 changes: 6 additions & 0 deletions a
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
apt update
apt install -y build-essential cmake valgrind
cmake . -B build -DCMAKE_CXX_STANDARD=${{ matrix.cxx-std }} -DDYLIB_BUILD_TESTS=ON -DDYLIB_WARNING_AS_ERRORS=ON
cmake --build build
cd build
valgrind --leak-check=full --show-leak-kinds=all --error-exitcode=1 ./unit_tests

0 comments on commit 8bc1eb1

Please sign in to comment.