From: Jason Gunthorpe <jgg@xxxxxxxxxxxx> AZP does this now and shows the results in a much clearer way. Signed-off-by: Jason Gunthorpe <jgg@xxxxxxxxxxxx> --- .travis.yml | 1 - buildlib/travis-checkpatch | 30 ------------------------------ 2 files changed, 31 deletions(-) delete mode 100755 buildlib/travis-checkpatch diff --git a/.travis.yml b/.travis.yml index 23226a679acb6b..d20dadf8e9d90f 100644 --- a/.travis.yml +++ b/.travis.yml @@ -48,7 +48,6 @@ before_script: - http_proxy= pip3 install cython script: - buildlib/travis-build - - buildlib/travis-checkpatch - buildlib/github-release deploy: # Deploy assets to Github releases diff --git a/buildlib/travis-checkpatch b/buildlib/travis-checkpatch deleted file mode 100755 index 5e78ec47406210..00000000000000 --- a/buildlib/travis-checkpatch +++ /dev/null @@ -1,30 +0,0 @@ -#!/bin/bash -# Copyright 2017 Mellanox Technologies Ltd. -# Licensed under BSD (MIT variant) or GPLv2. See COPYING. - - -if [ "x$TRAVIS_EVENT_TYPE" != "xpull_request" ]; then - # Peform checkpatch checks on pull requests only - exit 0 -fi - -# The below "set" is commented, because the checkpatch.pl returns 1 (error) for warnings too. -# And the rdma-core code is not mature enough to be warning safe -# set -e - -if [ "x$TRAVIS_COMMIT_RANGE" != "x" ]; then - cd buildlib/ - wget -q https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/plain/scripts/checkpatch.pl \ - https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/plain/scripts/spelling.txt - DIR_FOR_PATCHES_TO_CHECK=$(mktemp -d) - git format-patch --no-cover-letter $TRAVIS_COMMIT_RANGE ^$TRAVIS_BRANCH -o $DIR_FOR_PATCHES_TO_CHECK/ - CHECKPATCH_OPT="--no-tree --ignore PREFER_KERNEL_TYPES,FILE_PATH_CHANGES,EXECUTE_PERMISSIONS,USE_NEGATIVE_ERRNO,CONST_STRUCT $DIR_FOR_PATCHES_TO_CHECK/*" - perl checkpatch.pl $CHECKPATCH_OPT - if [ $? -ne 0 ]; then - # We rerun checkpatch to simplify parsing and to understand if we failed for errors - # For example, the output on some arbitrary patchset of the following line without awk is: - # total: 1 errors, 3 warnings, 42 lines checked - NUMB_ERRRORS=$(perl checkpatch.pl --terse $CHECKPATCH_OPT | awk 'BEGIN {FS = "total:"} ; {sum+=$2} END {print sum}') - exit $NUMB_ERRRORS - fi -fi -- 2.22.0
