return 0
}
+# test commit body for length
+function test_commit_bodylength()
+{
+ length="72"
+ body=$(git log -n 1 --pretty=%b "$REF" | egrep -m 1 ".{$((length + 1))}")
+ if [ -n "$body" ]; then
+ echo "error: commit message body contains line over ${length} characters"
+ return 1
+ fi
+
+ return 0
+}
+
# check for a tagged line
function check_tagged_line()
{
}
# check for a tagged line and check that the link is valid
-function check_tagged_line_with_url ()
+function check_tagged_line_with_url()
{
regex='^\s*'"$1"':\s\K([[:graph:]]+)$'
foundline=$(git log -n 1 "$REF" | grep -Po "$regex")
fi
# ensure that no lines in the body of the commit are over 72 characters
- body=$(git log -n 1 --pretty=%b "$REF" | egrep -m 1 '.{73}')
- if [ -n "$body" ]; then
- echo "error: commit message body contains line over 72 characters"
+ if ! test_commit_bodylength ; then
error=1
fi
function openzfs_port_commit()
{
+ error=0
+
# subject starts with OpenZFS dddd
subject=$(git log -n 1 --pretty=%s "$REF" | egrep -m 1 '^OpenZFS [[:digit:]]+ - ')
if [ -z "$subject" ]; then
- echo "OpenZFS patch ports must have a summary that starts with \"OpenZFS dddd - \""
+ echo "error: OpenZFS patch ports must have a subject line that starts with \"OpenZFS dddd - \""
error=1
fi
return $error
}
+function is_coverity_fix()
+{
+ # subject starts with Fix coverity defects means it's a coverity fix
+ subject=$(git log -n 1 --pretty=%s "$REF" | egrep -m 1 '^Fix coverity defects')
+ if [ -n "$subject" ]; then
+ return 0
+ fi
+
+ return 1
+}
+
+function coverity_fix_commit()
+{
+ error=0
+
+ # subject starts with Fix coverity defects: CID dddd, dddd...
+ subject=$(git log -n 1 --pretty=%s "$REF" |
+ egrep -m 1 'Fix coverity defects: CID [[:digit:]]+(, [[:digit:]]+)*')
+ if [ -z "$subject" ]; then
+ echo "error: Coverity defect fixes must have a subject line that starts with \"Fix coverity defects: CID dddd\""
+ error=1
+ fi
+
+ # need a signed off by
+ if ! check_tagged_line "Signed-off-by" ; then
+ error=1
+ fi
+
+ # test each summary line for the proper format
+ OLDIFS=$IFS
+ IFS=$'\n'
+ for line in $(git log -n 1 --pretty=%b "$REF" | egrep '^CID'); do
+ echo "$line" | egrep '^CID [[:digit:]]+: ([[:graph:]]+|[[:space:]])+ \(([[:upper:]]|\_)+\)' > /dev/null
+ if [[ $? -ne 0 ]]; then
+ echo "error: commit message has an improperly formatted CID defect line"
+ error=1
+ fi
+ done
+ IFS=$OLDIFS
+
+ # ensure that no lines in the body of the commit are over 72 characters
+ if ! test_commit_bodylength; then
+ error=1
+ fi
+
+ return $error
+}
+
if [ -n "$1" ]; then
REF="$1"
fi
fi
fi
+# if coverity fix, test against that
+if is_coverity_fix; then
+ if ! coverity_fix_commit; then
+ exit 1
+ else
+ exit 0
+ fi
+fi
+
# have a normal commit
if ! new_change_commit ; then
exit 1