Skip to content

Commit

Permalink
Split roundtrip_test and leakcheck.
Browse files Browse the repository at this point in the history
CI status became green even if roundtrip_test failed, since `then` block
was not suspended when $? was non-zero. Splitting roundtrip_test and
leakcheck addresses this problem.

With this change, builds checks ``${TRAVIS_OS_NAME}`` twice, but
checking a variable is cheap enough.

Signed-off-by: OGINO Masanori <[email protected]>
  • Loading branch information
omasanori committed Jul 16, 2015
1 parent 6f03902 commit 03f7314
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,10 @@ script:
- |
if [ ${TRAVIS_OS_NAME:-'linux'} = 'linux' ]
then
make roundtrip_test
make leakcheck
make roundtrip_test
fi
- |
if [ ${TRAVIS_OS_NAME:-'linux'} = 'linux' ]
then
make leakcheck
fi

0 comments on commit 03f7314

Please sign in to comment.