diff options
author | Loïc Blot <nerzhul@users.noreply.github.com> | 2017-03-26 14:07:07 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-03-26 14:07:07 +0200 |
commit | 72ce9d7a5d51f6308390748693e518632c1795e6 (patch) | |
tree | c71687b6335a09619d59285b41b83c31bbb8c283 | |
parent | aa5549ecc33606f786c8cef433e0dbe9188f3ede (diff) | |
download | minetest-72ce9d7a5d51f6308390748693e518632c1795e6.tar.gz minetest-72ce9d7a5d51f6308390748693e518632c1795e6.tar.bz2 minetest-72ce9d7a5d51f6308390748693e518632c1795e6.zip |
clang-format: fix counter increment & output (#5456)
clang-format: fix bash syntax on increment
-rwxr-xr-x | util/travis/script.sh | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/util/travis/script.sh b/util/travis/script.sh index ab524b4b4..0e4f92eef 100755 --- a/util/travis/script.sh +++ b/util/travis/script.sh @@ -4,6 +4,7 @@ needs_compile || exit 0 function perform_lint() { + echo "Performing LINT..." CLANG_FORMAT=clang-format-3.9 if [ "$TRAVIS_EVENT_TYPE" = "pull_request" ]; then # Get list of every file modified in this pull request @@ -17,13 +18,15 @@ function perform_lint() { local fail=0 for f in ${files_to_lint}; do d=$(diff -u "$f" <(${CLANG_FORMAT} "$f") || true) + if ! [ -z "$d" ]; then - ((errorcount++)) - printf "The file %s is not compliant with the coding style:\n%s\n" "$f" "$d" + errorcount=$((errorcount+1)) + + printf "The file %s is not compliant with the coding style" "$f" if [ ${errorcount} -gt 50 ]; then - printf "Too many errors encountered previously, this diff is hidden.\n" + printf "\nToo many errors encountered previously, this diff is hidden.\n" else - printf "%s\n" "$d" + printf ":\n%s\n" "$d" fi # Disable build failure at this moment as we need to have a complete MT source whitelist to check fail=0 @@ -31,15 +34,15 @@ function perform_lint() { done if [ "$fail" = 1 ]; then + echo "LINT reports failure." exit 1 fi - - exit 0 } if [[ "$LINT" == "1" ]]; then # Lint with exit CI perform_lint + exit 0 fi if [[ $PLATFORM == "Unix" ]]; then |