Repository: flink Updated Branches: refs/heads/master 57615aaa1 -> 4b53e66c4
[QA] display max 100 lines of compiler warnings hide warning if output file does not exist Project: http://git-wip-us.apache.org/repos/asf/flink/repo Commit: http://git-wip-us.apache.org/repos/asf/flink/commit/4b53e66c Tree: http://git-wip-us.apache.org/repos/asf/flink/tree/4b53e66c Diff: http://git-wip-us.apache.org/repos/asf/flink/diff/4b53e66c Branch: refs/heads/master Commit: 4b53e66c447d6f91b505da52eb4cd19a65640194 Parents: 57615aa Author: Maximilian Michels <m...@apache.org> Authored: Tue May 12 13:55:58 2015 +0200 Committer: Maximilian Michels <m...@apache.org> Committed: Tue May 12 14:18:46 2015 +0200 ---------------------------------------------------------------------- tools/qa-check.sh | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/flink/blob/4b53e66c/tools/qa-check.sh ---------------------------------------------------------------------- diff --git a/tools/qa-check.sh b/tools/qa-check.sh index 6881887..ea8f9f6 100755 --- a/tools/qa-check.sh +++ b/tools/qa-check.sh @@ -70,7 +70,7 @@ cd flink export TESTS_PASSED=true # Store output of results in a file in the qa dir QA_OUTPUT="$VAR_DIR/qa_results.txt" -rm "$QA_OUTPUT" +rm -f "$QA_OUTPUT" append_output() { echo "$1" @@ -115,7 +115,8 @@ checkCompilerWarnings() { if [ "$NEW_COMPILER_ERR_CNT" -gt "$OLD_COMPILER_ERR_CNT" ]; then append_output ":-1: The change increases the number of compiler warnings from $OLD_COMPILER_ERR_CNT to $NEW_COMPILER_ERR_CNT" append_output '```diff' - append_output `diff $VAR_DIR/_COMPILER_REFERENCE_WARNINGS $VAR_DIR/_COMPILER_NEW_WARNINGS` + append_output "First 100 warnings:" + append_output "`diff $VAR_DIR/_COMPILER_REFERENCE_WARNINGS $VAR_DIR/_COMPILER_NEW_WARNINGS | head -n 100`" append_output '```' TESTS_PASSED=false else