Skip to content

Commit

Permalink
ci: Enable sanitiziers by default
Browse files Browse the repository at this point in the history
These are cheap enough and valuable enough that we should
really run with them on by default in CI.

Signed-off-by: Colin Walters <walters@verbum.org>
  • Loading branch information
cgwalters committed Jun 22, 2023
1 parent 3e750ff commit 70e0db3
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ jobs:
- name: Configure
run: ./autogen.sh && ./configure --prefix=/usr --sysconfdir=/etc --libdir=/usr/lib/$(dpkg-architecture -qDEB_HOST_MULTIARCH) CFLAGS='-Wall -Werror'
- name: Build
run: make -j $(nproc)
run: make -j $(nproc) CFLAGS='-fsanitize=address -fsanitize=undefined'
- name: Unit tests
run: make check
- name: Capture build
Expand Down Expand Up @@ -71,6 +71,8 @@ jobs:
steps:
- name: Install erofs kmod
run: sudo apt install linux-modules-extra-$(uname -r)
- name: Install sanitizer dependencies
run: sudo apt install libasan6 libubsan1
- name: Checkout repository
uses: actions/checkout@v3
- name: Download
Expand Down

0 comments on commit 70e0db3

Please sign in to comment.