Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12471 )
Change subject: [build-support] KUDU-2699 introduce TIDY build type ...................................................................... [build-support] KUDU-2699 introduce TIDY build type This changelist introduces a new TIDY build type to facilitate running clang_tidy_gerrit.py from the kudu-tidy-bot Jenkins job by jenkins.kudu.apache.org. The idea is to make sure all auto-generated header files are present before running the clang-tidy tool on the Kudu C++ source files. As Adar noticed, our development process is such that new work goes into master and any commits to release branches are pure backports. There's not much point in running clang-tidy on those backports. So, the proposal is to simply run the kudu-tidy-bot job only on the changes in the master branch. That entails updating the configuration of the kudu-tidy-bot Jenkins job after this change is pushed into the repo. The corresponding command for the config for the kudu-tidy-bot job is: ---------------------------------------------------------------------- \#!/bin/bash export TIDYBOT_PASSWORD=<hidden> export PATH=/usr/lib/ccache:$PATH ccache -M 50G set -ex export BUILD_TYPE=TIDY build-support/jenkins/build-and-test.sh ---------------------------------------------------------------------- Change-Id: I9b364620c9fb92202cacf0286df292dd4cc9952b Reviewed-on: http://gerrit.cloudera.org:8080/12471 Reviewed-by: Adar Dembo <a...@cloudera.com> Tested-by: Kudu Jenkins --- M CMakeLists.txt M build-support/jenkins/build-and-test.sh 2 files changed, 28 insertions(+), 6 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/12471 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I9b364620c9fb92202cacf0286df292dd4cc9952b Gerrit-Change-Number: 12471 Gerrit-PatchSet: 7 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120)
