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

Commit 0ec2c657 authored by Ian Maund's avatar Ian Maund
Browse files

Revert "checkpatch: emit a warning on file add/move/delete" 



This reverts commit 13f1937e
("checkpatch: emit a warning on file add/move/delete")
We do not use a maintainers file for tracking file owners/maintainers.
Remove this check.

Change-Id: If090c1c0b5fc16d87cd5342b57e3bdfde19a7fd4
Signed-off-by: default avatarIan Maund <imaund@codeaurora.org>
parent 1c2e92c6

scripts/checkpatch.pl

+1 −12
Original line number Diff line number Diff line
@@ -1878,7 +1878,7 @@ sub process { 


	my $in_header_lines = $file ? 0 : 1;

	my $in_commit_log = 0;		#Scanning lines before patch

	my $reported_maintainer_file = 0;



	my $non_utf8_charset = 0;



	my $last_blank_line = 0;
@@ -2321,17 +2321,6 @@ sub process { 
			      "Please use 12 or more chars for the git commit ID and enclose the descritpion in parenthesis like: '${init_char}ommit $id (\"$desc\")'\n" . $herecurr);

		}



# Check for added, moved or deleted files

		if (!$reported_maintainer_file && !$in_commit_log &&

		    ($line =~ /^(?:new|deleted) file mode\s*\d+\s*$/ ||

		     $line =~ /^rename (?:from|to) [\w\/\.\-]+\s*$/ ||

		     ($line =~ /\{\s*([\w\/\.\-]*)\s*\=\>\s*([\w\/\.\-]*)\s*\}/ &&

		      (defined($1) || defined($2))))) {

			$reported_maintainer_file = 1;

			WARN("FILE_PATH_CHANGES",

			     "added, moved or deleted file(s), does MAINTAINERS need updating?\n" . $herecurr);

		}



#check the patch for invalid author credentials

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

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