Skip to content

Commit

Permalink
.github: Enable cdefs tests for build-only targets
Browse files Browse the repository at this point in the history
We can at least validate the include files during CI.

Signed-off-by: Keith Packard <keithp@keithp.com>
  • Loading branch information
keith-packard committed Jul 26, 2024
1 parent 3f13cf3 commit f494a5d
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions .github/do-build
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,9 @@ done

ANALYZER="-Danalyzer=true"

CDEFS="-Dtests-cdefs=true"
TEST="test"

# Disable analyzer on m68k because the compiler gets very confused
# about complex values

Expand All @@ -32,9 +35,9 @@ mkdir "$DIR"
trap 'rm -rf "$DIR"' 0 1 15
(cd "$DIR" || exit 1
echo '##################################################'
echo '##########' ../scripts/"$CONFIGURE" -Dwerror=true $ANALYZER "$@"
echo '##########' ../scripts/"$CONFIGURE" -Dwerror=true $ANALYZER $CDEFS "$@"
echo '##################################################'
../scripts/"$CONFIGURE" -Dwerror=true $ANALYZER "$@"
../scripts/"$CONFIGURE" -Dwerror=true $ANALYZER $CDEFS "$@"
case $? in
0)
echo 'Configuration succeeded'
Expand All @@ -50,4 +53,4 @@ trap 'rm -rf "$DIR"' 0 1 15
;;
esac
cat meson-logs/*
ninja || exit 1) || exit 1
ninja $TEST || exit 1) || exit 1

0 comments on commit f494a5d

Please sign in to comment.