Skip to content

Commit

Permalink
workflows: Move gtk-doc enablement from clang to valgrind build
Browse files Browse the repository at this point in the history
We want to have gtk-doc enabled in the build that will run
`make distcheck`, but the clang/CodeQL build seems like a poor fit for
that, since it runs twice (for C and Python) and has extra
instrumentation. Move it to the build where we will run tests under
valgrind, which is already somewhat slow.

Signed-off-by: Simon McVittie <[email protected]>
  • Loading branch information
smcv authored and alexlarsson committed May 3, 2022
1 parent cbb6176 commit cb73e09
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ jobs:
with:
submodules: true
- name: configure
run: ./autogen.sh --enable-gtk-doc --enable-gtk-doc-html
run: ./autogen.sh
env:
CC: clang
CFLAGS: -Werror=unused-variable
Expand Down Expand Up @@ -172,7 +172,7 @@ jobs:
run: |
mkdir _build
pushd _build
../configure
../configure --enable-gtk-doc --enable-gtk-doc-html
popd
env:
CFLAGS: -O2
Expand Down

0 comments on commit cb73e09

Please sign in to comment.