Skip to content

Commit 0093e32

Browse files
authored
[ci] Skip valgrind checks on OS X
1 parent 9e091f3 commit 0093e32

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

.github/workflows/ci.yml

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -752,12 +752,16 @@ jobs:
752752
declare -i RETCODE=0
753753
754754
set -o pipefail
755-
if [[ "${{ matrix.clang-runtime }}" == "17" || "${{ matrix.clang-runtime }}" == "18" ]] && [[ "${{ matrix.os }}" != "macos-14" ]]; then
756-
echo "Valgrind reports true for clang-runtime 17 or 18, due to memory leaks with LLVM"
757-
valgrind --error-exitcode=1 --suppressions=../etc/valgrind-cppyy-cling.supp python -m pytest -m "not xfail" -v || true
755+
if [[ "${{ matrix.os }}" == macos-* ]]; then
756+
echo "Skipping Valgrind checks on macOS"
758757
else
759-
echo "Running valgrind on passing tests"
760-
valgrind --error-exitcode=1 --suppressions=../etc/valgrind-cppyy-cling.supp python -m pytest -m "not xfail" -v
758+
if [[ "${{ matrix.clang-runtime }}" == "17" || "${{ matrix.clang-runtime }}" == "18" ]]; then
759+
echo "Valgrind reports true for clang-runtime 17 or 18, due to memory leaks with LLVM"
760+
valgrind --error-exitcode=1 --suppressions=../etc/valgrind-cppyy-cling.supp python -m pytest -m "not xfail" -v || true
761+
else
762+
echo "Running valgrind on passing tests"
763+
valgrind --error-exitcode=1 --suppressions=../etc/valgrind-cppyy-cling.supp python -m pytest -m "not xfail" -v
764+
fi
761765
fi
762766
export RETCODE=+$?
763767
echo ::endgroup::

0 commit comments

Comments
 (0)