Skip to content

Commit

Permalink
ci/linux: enable leak detector
Browse files Browse the repository at this point in the history
The leak detector has been disabled due to a libass leak that was
causing the test to fail. I initially expected this change to be
temporary until a patch release of libass became available and we could
update it. However, it has been a few months without a release. Instead
of waiting, re-enable the leak detector and add a suppression file for
this specific leak.

Fixes: d6eb85b
  • Loading branch information
kasper93 committed Jan 26, 2025
1 parent 964017c commit feef1bc
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -290,7 +290,6 @@ jobs:
env:
CC: "${{ matrix.config.cc }}"
CXX: "${{ matrix.config.cxx }}"
ASAN_OPTIONS: "halt_on_error=1:abort_on_error=1:print_summary=1:detect_leaks=0"
strategy:
matrix:
config:
Expand Down Expand Up @@ -318,6 +317,7 @@ jobs:
- name: Run meson tests
id: tests
run: |
export LSAN_OPTIONS="suppressions=${GITHUB_WORKSPACE}/.lsan_suppressions"
meson test -C build
- name: Print meson test log
Expand Down
1 change: 1 addition & 0 deletions .lsan_suppressions
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
leak:FcFontSetSort

0 comments on commit feef1bc

Please sign in to comment.