[PATCH] get_maintainer: Don't spam others in fallback mode if there is a maintainer

[PATCH] get_maintainer: Don't spam others in fallback mode if there is a maintainer

[Thomas Huth]

From: Thomas Huth <thuth@redhat.com>

The get_maintainer.pl script currently suggests to CC: people who
contributed to a subfolder of a maintained subsystem, e.g.:

 $ scripts/get_maintainer.pl -f arch/arm64/include/asm/smp.h
 Catalin Marinas <catalin.marinas@arm.com> (maintainer:ARM64 PORT (AARCH64 ARCHITECTURE),commit_signer:1/1=100%)
 Will Deacon <will@kernel.org> (maintainer:ARM64 PORT (AARCH64 ARCHITECTURE))
 Thomas Huth <thuth@redhat.com> (commit_signer:1/1=100%,authored:1/1=100%,added_lines:2/2=100%,removed_lines:2/2=100%)
 linux-arm-kernel@lists.infradead.org (moderated list:ARM64 PORT (AARCH64 ARCHITECTURE))
 linux-kernel@vger.kernel.org (open list)

So if you contributed a single line to a subfolder of a maintained
and active subsystem, you get regularly CC:-ed for all other changes
to the corresponding file during the next year. That's quite excessive.

Limit the fallback mode to files that are not maintained at all by
removing the check for subfolders in the corresponding part of the code.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 scripts/get_maintainer.pl | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index 16b80a700d4ac..ded73b9673eb9 100755
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -938,8 +938,7 @@ sub get_maintainers {
 				my $file_pd = ($file  =~ tr@/@@);
 				$value_pd++ if (substr($value,-1,1) ne "/");
 				$value_pd = -1 if ($value =~ /^(\.\*|\(\?:\.\*\))/);
-				if ($value_pd >= $file_pd &&
-				    range_is_maintained($start, $end) &&
+				if (range_is_maintained($start, $end) &&
 				    range_has_maintainer($start, $end)) {
 				    $exact_pattern_match_hash{$file} = 1;
 				}
-- 
2.55.0