Skip to content

Commit

Permalink
ci: workaround makefile dep ordering issue
Browse files Browse the repository at this point in the history
`make -j all check_all` has a race condition that runs lex/yacc twice,
and sometimes causes the build to fail.

Signed-off-by: Robin H. Johnson <[email protected]>
  • Loading branch information
robbat2 committed Jan 2, 2025
1 parent 10ce922 commit a8c0f4c
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion .github/workflows/codeql.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,8 @@ jobs:
DEBIAN_FRONTEND=noninteractive sudo apt-get -qy install autoconf automake bison build-essential check clang flex gettext libtool pkg-config libbsd-dev libbsd0 systemd || exit 1
./autogen.sh || exit 1
./configure --with-check || exit 1
make -j all check_all || exit 1
make -j all || exit 1 # If both all & check_all are together, triggers race condition w/ scanner.l & gram.y
make -j check_all || exit 1
make check || exit 1
- name: Perform CodeQL Analysis
Expand Down

0 comments on commit a8c0f4c

Please sign in to comment.