summaryrefslogtreecommitdiffstats
path: root/rfc.sh
diff options
context:
space:
mode:
authorJustin Clift <justin@gluster.org>2015-04-12 16:23:55 +0100
committerVijay Bellur <vbellur@redhat.com>2015-04-13 11:10:07 +0000
commitdb5fc47e2022a18038e3877f69a708532b64969c (patch)
tree51ebb1b5c1ed45735100524ffb06d13fb044ff9c /rfc.sh
parent0333ac8abf9d5d1cc95fea80fba098c7d2c4c8c3 (diff)
Fix incorrect rfc.sh and checkpatch.pl use of "jenkins" instead of "gerrit"
Change-Id: I6e78477554097a2be5aad8e3fb196970fb30dac2 Signed-off-by: Justin Clift <justin@gluster.org> Reviewed-on: http://review.gluster.org/10203 Tested-by: NetBSD Build System Reviewed-by: Niels de Vos <ndevos@redhat.com> Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
Diffstat (limited to 'rfc.sh')
-rwxr-xr-xrfc.sh10
1 files changed, 6 insertions, 4 deletions
diff --git a/rfc.sh b/rfc.sh
index 5528629c7fe..287768eae8b 100755
--- a/rfc.sh
+++ b/rfc.sh
@@ -82,6 +82,7 @@ assert_diverge()
git diff origin/$branch..HEAD | grep -q .;
}
+
check_patches_for_coding_style()
{
git fetch origin;
@@ -92,8 +93,8 @@ check_patches_for_coding_style()
exit 1
fi
- ## Set this to known value once Jenkins URL changes
- export JENKINS_URL="review.gluster.org"
+ # The URL of our Gerrit server
+ export GERRIT_URL="review.gluster.org"
echo "Running coding guidelines check ..."
head=$(git rev-parse --abbrev-ref HEAD)
@@ -104,13 +105,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 --jenkins-url ${JENKINS_URL} -"
+ echo " | ./extras/checkpatch.pl --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 --jenkins-url ${JENKINS_URL} -"
+ echo " | ./extras/checkpatch.pl --gerrit-url ${GERRIT_URL} -"
echo -n "Do you want to continue anyway [no/yes]: "
read yesno
if [ "${yesno}" != "yes" ] ; then
@@ -120,6 +121,7 @@ check_patches_for_coding_style()
fi
}
+
main()
{
set_hooks_commit_msg;