[coreboot-gerrit] Patch set updated for coreboot: util/lint: Add check for the signed-off-by line

Martin Roth (martinroth@google.com) gerrit at coreboot.org
Fri Jan 6 02:57:44 CET 2017


Martin Roth (martinroth at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17941

-gerrit

commit fae06a54e6d4085c7e5e27429289ef0d357e712d
Author: Martin Roth <martinroth at google.com>
Date:   Thu Dec 22 10:54:55 2016 -0700

    util/lint: Add check for the signed-off-by line
    
    Gerrit will let you submit a patch without a signed-off-by line,
    although I believe it can't actually be merged.  Instead of catching
    it either manually, or when the patch is attempting to be merged,
    catch this in the jenkins builder.
    
    Change-Id: I80161befa157266dd4e3209839a06ff398aab6bb
    Signed-off-by: Martin Roth <martinroth at google.com>
---
 util/lint/lint-stable-020-signed-off-by | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/util/lint/lint-stable-020-signed-off-by b/util/lint/lint-stable-020-signed-off-by
new file mode 100755
index 0000000..40b6e9e
--- /dev/null
+++ b/util/lint/lint-stable-020-signed-off-by
@@ -0,0 +1,23 @@
+#!/bin/sh
+# This file is part of the coreboot project.
+#
+# Copyright 2016 Google Inc.
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; version 2 of the License, or (at your option)
+# any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# DESCR: Check for a signed-off-by line on the latest git commit
+
+# This test is mainly for the jenkins server
+if [ -n "$(command -v git)" ] && [ -d .git ]; then
+	if [ -z "$(git log -n 1 | grep '[[:space:]]\+Signed-off-by: ')" ]; then
+		echo "No Signed-off-by line in commit message"
+	fi
+fi



More information about the coreboot-gerrit mailing list