From 03f73146e79981bb5cf23695c46235e688a32541 Mon Sep 17 00:00:00 2001 From: OGINO Masanori Date: Thu, 16 Jul 2015 08:50:27 +0900 Subject: Split roundtrip_test and leakcheck. 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 --- .travis.yml | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/.travis.yml b/.travis.yml index 5de5632..5ffc63b 100644 --- a/.travis.yml +++ b/.travis.yml @@ -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 -- cgit v1.2.3 From 16374c5e66fe44774665b33120b98526f5499940 Mon Sep 17 00:00:00 2001 From: OGINO Masanori Date: Thu, 16 Jul 2015 09:07:53 +0900 Subject: Fix indentations. Signed-off-by: OGINO Masanori --- .travis.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.travis.yml b/.travis.yml index 5ffc63b..25c3ec1 100644 --- a/.travis.yml +++ b/.travis.yml @@ -29,10 +29,10 @@ script: - | if [ ${TRAVIS_OS_NAME:-'linux'} = 'linux' ] then - make roundtrip_test + make roundtrip_test fi - | if [ ${TRAVIS_OS_NAME:-'linux'} = 'linux' ] then - make leakcheck + make leakcheck fi -- cgit v1.2.3