Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2d8d1851 authored by Jeff Johnson's avatar Jeff Johnson Committed by Kyle Yan
Browse files

checkpatch: add support for --noauthor 



Commit "scripts: checkpatch.pl: warn on invalid credentials" added
to checkpatch.pl logic to unconditionally check for invalid author
credentials.  Add a flag to allow override of this behavior.

Change-Id: Ib254cc6f73c4de857ba923e6170750262cde01ad
Signed-off-by: default avatarJeff Johnson <jjohnson@codeaurora.org>
parent f2dde159

scripts/checkpatch.pl

+5 −2
Original line number Diff line number Diff line
@@ -22,6 +22,7 @@ my $quiet = 0; 
my $tree = 1;

my $chk_signoff = 1;

my $chk_patch = 1;

my $chk_author = 1;

my $tst_only;

my $emacs = 0;

my $terse = 0;
@@ -64,6 +65,7 @@ Options: 
  -q, --quiet                quiet

  --no-tree                  run without a kernel tree

  --no-signoff               do not check for 'Signed-off-by' line

  --no-author                do not check for unexpected authors

  --patch                    treat FILE as patchfile (default)

  --emacs                    emacs compile window format

  --terse                    one line per report
@@ -137,6 +139,7 @@ GetOptions( 
	'tree!'		=> \$tree,

	'signoff!'	=> \$chk_signoff,

	'patch!'	=> \$chk_patch,

	'author!'	=> \$chk_author,

	'emacs!'	=> \$emacs,

	'terse!'	=> \$terse,

	'showfile!'	=> \$showfile,
@@ -2335,7 +2338,7 @@ sub process { 
					     "email address '$email' might be better as '$suggested_email$comment'\n" . $herecurr);

				}

			}

			if ($line =~ /^\s*signed-off-by:.*(quicinc|qualcomm)\.com/i) {

			if ($chk_author && $line =~ /^\s*signed-off-by:.*(quicinc|qualcomm)\.com/i) {

				WARN("BAD_SIGN_OFF",

				     "invalid Signed-off-by identity\n" . $line );

			}
@@ -2470,7 +2473,7 @@ sub process { 
		}



#check the patch for invalid author credentials

		if ($line =~ /^From:.*(quicinc|qualcomm)\.com/) {

		if ($chk_author && $line =~ /^From:.*(quicinc|qualcomm)\.com/) {

			WARN("BAD_AUTHOR", "invalid author identity\n" . $line );

		}