Skip to content

Commit

Permalink
Build chkjson on Travis
Browse files Browse the repository at this point in the history
Merging CleverRaven#30759 broke the build by introducing a compile failure in
chkjson.  This should be built on Travis to allow detection of such
issues.

Previously chkjson was only built on Jenkins, but that doesn't happen
when the Makefile changes, so this problem went undetected.
  • Loading branch information
jbytheway committed May 24, 2019
1 parent 58fdd83 commit 419ce85
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions build-scripts/build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,10 @@ if [ -n "$TEST_STAGE" ]
then
build-scripts/lint-json.sh
make -j "$num_jobs" style-json

# Also build chkjson (even though we're not using it), to catch any
# compile errors there
make -j "$num_jobs" chkjson
elif [ -n "$JUST_JSON" ]
then
echo "Early exit on just-json change"
Expand Down

0 comments on commit 419ce85

Please sign in to comment.