From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752298AbdLGA7s (ORCPT ); Wed, 6 Dec 2017 19:59:48 -0500 Received: from out5-smtp.messagingengine.com ([66.111.4.29]:34413 "EHLO out5-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752160AbdLGA7r (ORCPT ); Wed, 6 Dec 2017 19:59:47 -0500 X-ME-Sender: From: "Tobin C. Harding" To: Andy Whitcroft , Joe Perches Cc: "Tobin C. Harding" , linux-kernel@vger.kernel.org, Greg Kroah-Hartman Subject: [PATCH] checkpatch: add check for tag Co-Developed-by Date: Thu, 7 Dec 2017 11:59:36 +1100 Message-Id: <1512608376-4234-1-git-send-email-me@tobin.cc> X-Mailer: git-send-email 2.7.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Recently signature tag Co-Developed-by was added to the kernel (Documentation/process/5.Posting.rst). checkpatch.pl doesn't know about it yet. All prior tags used all lowercase characters except for first character. Checks for this format had to be re-worked to allow for the new tag. Add checkpatch checks for Co-Developed-by tag. Cc: Greg Kroah-Hartman Signed-off-by: Tobin C. Harding --- scripts/checkpatch.pl | 22 ++++++++++++++++------ 1 file changed, 16 insertions(+), 6 deletions(-) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 040aa79e1d9d..a7d2cdcec6a6 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -468,6 +468,7 @@ our $signature_tags = qr{(?xi: Reviewed-by:| Reported-by:| Suggested-by:| + Co-Developed-by:| To:| Cc: )}; @@ -2468,6 +2469,8 @@ sub process { my $space_after = $3; my $email = $4; my $ucfirst_sign_off = ucfirst(lc($sign_off)); + my $preferred_signature = ""; + my $co_dev_by_tag = 'Co-Developed-by:'; if ($sign_off !~ /$signature_tags/) { WARN("BAD_SIGN_OFF", @@ -2481,15 +2484,22 @@ sub process { "$ucfirst_sign_off $email"; } } - if ($sign_off =~ /-by:$/i && $sign_off ne $ucfirst_sign_off) { - if (WARN("BAD_SIGN_OFF", - "'$ucfirst_sign_off' is the preferred signature form\n" . $herecurr) && - $fix) { - $fixed[$fixlinenr] = - "$ucfirst_sign_off $email"; + if ($sign_off =~ /$co_dev_by_tag$/i) { + if ($sign_off ne $co_dev_by_tag) { + $preferred_signature = $co_dev_by_tag; } + } elsif ($sign_off =~ /-by:$/i && $sign_off ne $ucfirst_sign_off) { + $preferred_signature = $ucfirst_sign_off; } + + if ($preferred_signature ne "") { + my $prefix = "'$preferred_signature' is the preferred signature form\n"; + if (WARN("BAD_SIGN_OFF", $prefix . $herecurr) && $fix) { + $fixed[$fixlinenr] = "$ucfirst_sign_off $email"; + } + } + if (!defined $space_after || $space_after ne " ") { if (WARN("BAD_SIGN_OFF", "Use a single space after $ucfirst_sign_off\n" . $herecurr) && -- 2.7.4