Subject: [merged] checkpatchpl-add-check-for-change-id.patch removed from -mm tree
To: cov@xxxxxxxxxxxxxx,apw@xxxxxxxxxxxx,joe@xxxxxxxxxxx,mm-commits@xxxxxxxxxxxxxxx
From: akpm@xxxxxxxxxxxxxxxxxxxx
Date: Fri, 04 Apr 2014 12:34:28 -0700
The patch titled
Subject: checkpatch.pl: add check for Change-Id
has been removed from the -mm tree. Its filename was
checkpatchpl-add-check-for-change-id.patch
This patch was dropped because it was merged into mainline or a subsystem tree
------------------------------------------------------
From: Christopher Covington <cov@xxxxxxxxxxxxxx>
Subject: checkpatch.pl: add check for Change-Id
A commit hook for the Gerrit code review server [1] inserts change
identifiers so Gerrit can track patches through multiple revisions. These
identifiers are noise in the context of the upstream kernel. (Many Gerrit
servers are private. Even given a public instance, given only a
Change-Id, one must guess which server a change was tracked on. Patches
submitted to the Linux kernel mailing lists should be able to stand on
their own. If it's truly useful to reference code review on a Gerrit
server, a URL is a much clearer way to do so.) Thus, issue an error when a
Change-Id line is encountered before the Signed-off-by.
1. https://gerrit.googlesource.com/gerrit/+/master/gerrit-server/src/main/resources/com/google/gerrit/server/tools/root/hooks/commit-msg
Signed-off-by: Christopher Covington <cov@xxxxxxxxxxxxxx>
Cc: Joe Perches <joe@xxxxxxxxxxx>
Cc: Andy Whitcroft <apw@xxxxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---
scripts/checkpatch.pl | 6 ++++++
1 file changed, 6 insertions(+)
diff -puN scripts/checkpatch.pl~checkpatchpl-add-check-for-change-id scripts/checkpatch.pl
--- a/scripts/checkpatch.pl~checkpatchpl-add-check-for-change-id
+++ a/scripts/checkpatch.pl
@@ -1924,6 +1924,12 @@ sub process {
}
}
+# Check for unwanted Gerrit info
+ if ($in_commit_log && $line =~ /^\s*change-id:/i) {
+ ERROR("GERRIT_CHANGE_ID",
+ "Remove Gerrit Change-Id's before submitting upstream.\n" . $herecurr);
+ }
+
# Check for wrappage within a valid hunk of the file
if ($realcnt != 0 && $line !~ m{^(?:\+|-| |\\ No newline|$)}) {
ERROR("CORRUPTED_PATCH",
_
Patches currently in -mm which might be from cov@xxxxxxxxxxxxxx are
origin.patch
linux-next.patch
--
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
