diff options
author | John Heroy <johnheroy@gmail.com> | 2015-10-21 14:23:46 -0700 |
---|---|---|
committer | John Heroy <johnheroy@gmail.com> | 2015-10-21 14:23:46 -0700 |
commit | 7a3851f2806a061c5ee030ddaed65876ea908e1c (patch) | |
tree | 2e7b8f391204a12b7eff0ee06d3d7681f3cacc90 /scripts/check-lint-count.bash | |
parent | 9fdcd2b1ce30b456158efe5e12876eb4e445a4c2 (diff) | |
parent | b7f00efd50a7f13156525807a0925e26997bc541 (diff) | |
download | connectbot-7a3851f2806a061c5ee030ddaed65876ea908e1c.tar.gz connectbot-7a3851f2806a061c5ee030ddaed65876ea908e1c.tar.bz2 connectbot-7a3851f2806a061c5ee030ddaed65876ea908e1c.zip |
Merge branch 'master' into volume-seekbar
Diffstat (limited to 'scripts/check-lint-count.bash')
-rwxr-xr-x | scripts/check-lint-count.bash | 57 |
1 files changed, 57 insertions, 0 deletions
diff --git a/scripts/check-lint-count.bash b/scripts/check-lint-count.bash new file mode 100755 index 0000000..c02f70e --- /dev/null +++ b/scripts/check-lint-count.bash @@ -0,0 +1,57 @@ +#!/usr/bin/env bash +# +# Checks the number of lint issues against historical values. Used in +# Travis CI builds to fail when the number increases by exploiting the +# caching mechanism. + +if [[ $# != 2 || ! -f $1 ]]; then \ + echo "Usage: $0 <lint.xml file> <historical.xml file>" + exit 1 +fi + +lint_file="$1" +historical_file="$2" + +xqilla="$(which xqilla)" + +if [[ ! -x $xqilla ]]; then \ + echo "Error: cannot find xqilla" + exit 1 +fi + +if [[ ! -f $historical_file ]]; then \ + # no cache history, store this one and exit + cp "$lint_file" "$historical_file" + exit 0 +fi + +tmp_dir="$(mktemp -d lint.XXXXXXXX)" +trap 'rm -rf $tmp_dir' ERR EXIT + +lint_results="$tmp_dir/lint.txt" +hist_results="$tmp_dir/hist.txt" + +run_query() { + local xqilla_script='string-join(//issue/location/(concat("file=", @file, " line=", @line, " column=", @column, " reason=", ../@summary)), " ")' + xqilla -i "$1" <(echo "$xqilla_script") | sed "s,$PWD/,,g" > "$2" +} + +run_query "$lint_file" "$lint_results" +run_query "$historical_file" "$hist_results" + +old_count=$(wc -l < "$hist_results") +new_count=$(wc -l < "$lint_results") + +echo "Historical count: $old_count, new count: $new_count" + +if [[ $new_count > $old_count ]]; then \ + echo "FAILURE: lint issues increased from $old_count to $new_count" + diff -u "$hist_results" "$lint_results" + exit 2 +fi + +if [[ $TRAVIS_PULL_REQUEST == false ]]; then \ + # Okay, we either stayed the same or reduced our number. + # Write it out so we can check it next build! + cp "$lint_file" "$historical_file" +fi |