andy/glusterfs
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

build: move checkpatch.pl to build-aux

Browse Source 
There is no need for checkpatch.pl to be in extras/, it is used similar
to other scripts that are placed in build-aux/.

Change-Id: Id0f4db1666d83dad31687aa7aadae128fc14c016
BUG: 1198849
Signed-off-by: Niels de Vos <ndevos@redhat.com>
Reviewed-on: http://review.gluster.org/15400
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
This commit is contained in:
Niels de Vos 2016-09-05 11:39:10 +02:00
parent c4e9ec653c
commit 00d9abd401
2 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

0
extras/checkpatch.pl → build-aux/checkpatch.pl
View File

10
rfc.sh
View File

@ -100,9 +100,9 @@ check_patches_for_coding_style()
{
git fetch origin;
check_patch_script=./extras/checkpatch.pl
if [ ! -e ./extras/checkpatch.pl ] ; then
echo "checkpatch is not executable .. abort"
check_patch_script=./build-aux/checkpatch.pl
if [ ! -e ${check_patch_script} ] ; then
echo "${check_patch_script} is not executable .. abort"
exit 1
fi
@ -118,13 +118,13 @@ check_patches_for_coding_style()
if [ "$RES" -eq 1 ] ; then
echo "Errors caught, get details by:"
echo " git format-patch --stdout origin/${branch}..${head} \\"
echo " | ./extras/checkpatch.pl --gerrit-url ${GERRIT_URL} -"
echo " | ${check_patch_script} --gerrit-url ${GERRIT_URL} -"
echo "and correct errors"
exit 1
elif [ "$RES" -eq 2 ] ; then
echo "Warnings caught, get details by:"
echo " git format-patch --stdout origin/${branch}..${head} \\"
echo " | ./extras/checkpatch.pl --gerrit-url ${GERRIT_URL} -"
echo " | ${check_patch_script} --gerrit-url ${GERRIT_URL} -"
echo -n "Do you want to continue anyway [no/yes]: "
read yesno
if [ "${yesno}" != "yes" ] ; then