Hi Joe, > -----Original Message----- > From: linux-rdma-owner@xxxxxxxxxxxxxxx [mailto:linux-rdma- > owner@xxxxxxxxxxxxxxx] On Behalf Of Parav Pandit > Sent: Tuesday, October 10, 2017 5:44 PM > To: joe@xxxxxxxxxxx; apw@xxxxxxxxxxxxx > Cc: linux-kernel@xxxxxxxxxxxxxxx; linux-rdma@xxxxxxxxxxxxxxx; Parav Pandit > <parav@xxxxxxxxxxxx>; Brad Erickson <bradley@xxxxxxxxxxxx> > Subject: [PATCH] checkpatch: Introduce check for format of Fixes line in commit > log > > This patch introduces a format check for 'Fixes' line in commit log for 12 > characters commit id and format for Fixes as ("..."). > > Its created against linux-rdma Doug's tree [1]. > > [1] git://git.kernel.org/pub/scm/linux/kernel/git/dledford/rdma.git > > Signed-off-by: Parav Pandit <parav@xxxxxxxxxxxx> > Signed-off-by: Brad Erickson <bradley@xxxxxxxxxxxx> > --- > scripts/checkpatch.pl | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index > dd2c262..7d933e4 100755 > --- a/scripts/checkpatch.pl > +++ b/scripts/checkpatch.pl > @@ -2548,6 +2548,14 @@ sub process { > "Remove Gerrit Change-Id's before submitting > upstream.\n" . $herecurr); > } > > +# Check for incorrect format for Fixes line in commit log > + if ($in_commit_log && $line =~ /^\s*Fixes:/i) { > + if ($line !~ /^\s*Fixes: [a-z0-9]{12} \(\".*?\"\)$/i) { > + ERROR("FIXES_FORMAT", > + "Follow format of Fixes: <12 characters > commit id> (\"...\")\n" . $herecurr); > + } > + } > + > # Check if the commit log is in a possible stack dump > if ($in_commit_log && !$commit_log_possible_stack_dump && > ($line =~ /^\s*(?:WARNING:|BUG:)/ || > -- Did you get a chance to review this minor improvement patch? Does it look ok? Can it be pushed to Linux-rdma tree? -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
