scripts/get_maintainer.pl: don't search MAINTAINERS for keywords or emails

[net-next-2.6.git] / scripts / get_maintainer.pl

diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index b2281982f52f1a156f3eb5b321d3ec9a5fcaee0f..77f4f2e4cd81183d769c14e1778fb31a1c99cf98 100755 (executable)
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -13,7 +13,7 @@
 use strict;
 
 my $P = $0;
-my $V = '0.24';
+my $V = '0.25';
 
 use Getopt::Long qw(:config no_auto_abbrev);
 
@@ -88,6 +88,7 @@ my %VCS_cmds_git = (
     "available" => '(which("git") ne "") && (-d ".git")',
     "find_signers_cmd" => "git log --no-color --since=\$email_git_since -- \$file",
     "find_commit_signers_cmd" => "git log --no-color -1 \$commit",
+    "find_commit_author_cmd" => "git log -1 --format=\"%an <%ae>\" \$commit",
     "blame_range_cmd" => "git blame -l -L \$diff_start,+\$diff_length \$file",
     "blame_file_cmd" => "git blame -l \$file",
     "commit_pattern" => "^commit [0-9a-f]{40,40}",
@@ -101,6 +102,7 @@ my %VCS_cmds_hg = (
        "hg log --date=\$email_hg_since" .
                " --template='commit {node}\\n{desc}\\n' -- \$file",
     "find_commit_signers_cmd" => "hg log --template='{desc}\\n' -r \$commit",
+    "find_commit_author_cmd" => "hg log -l 1 --template='{author}\\n' -r \$commit",
     "blame_range_cmd" => "",           # not supported
     "blame_file_cmd" => "hg blame -c \$file",
     "commit_pattern" => "^commit [0-9a-f]{40,40}",
@@ -302,7 +304,7 @@ foreach my $file (@ARGV) {
     }
     if ($from_filename) {
        push(@files, $file);
-       if (-f $file && ($keywords || $file_emails)) {
+       if ($file ne "MAINTAINERS" && -f $file && ($keywords || $file_emails)) {
            open(my $f, '<', $file)
                or die "$P: Can't open $file: $!\n";
            my $text = do { local($/) ; <$f> };
@@ -418,23 +420,23 @@ foreach my $file (@files) {
 
     foreach my $line (sort {$hash{$b} <=> $hash{$a}} keys %hash) {
        add_categories($line);
-           if ($sections) {
-               my $i;
-               my $start = find_starting_index($line);
-               my $end = find_ending_index($line);
-               for ($i = $start; $i < $end; $i++) {
-                   my $line = $typevalue[$i];
-                   if ($line =~ /^[FX]:/) {            ##Restore file patterns
-                       $line =~ s/([^\\])\.([^\*])/$1\?$2/g;
-                       $line =~ s/([^\\])\.$/$1\?/g;   ##Convert . back to ?
-                       $line =~ s/\\\./\./g;           ##Convert \. to .
-                       $line =~ s/\.\*/\*/g;           ##Convert .* to *
-                   }
-                   $line =~ s/^([A-Z]):/$1:\t/g;
-                   print("$line\n");
+       if ($sections) {
+           my $i;
+           my $start = find_starting_index($line);
+           my $end = find_ending_index($line);
+           for ($i = $start; $i < $end; $i++) {
+               my $line = $typevalue[$i];
+               if ($line =~ /^[FX]:/) {                ##Restore file patterns
+                   $line =~ s/([^\\])\.([^\*])/$1\?$2/g;
+                   $line =~ s/([^\\])\.$/$1\?/g;       ##Convert . back to ?
+                   $line =~ s/\\\./\./g;               ##Convert \. to .
+                   $line =~ s/\.\*/\*/g;               ##Convert .* to *
                }
-               print("\n");
+               $line =~ s/^([A-Z]):/$1:\t/g;
+               print("$line\n");
            }
+           print("\n");
+       }
     }
 
     if ($email && $email_git) {
@@ -1014,6 +1016,9 @@ sub vcs_find_signers {
     if (!$email_git_penguin_chiefs) {
        @lines = grep(!/${penguin_chiefs}/i, @lines);
     }
+
+    return (0, @lines) if !@lines;
+
     # cut -f2- -d":"
     s/.*:\s*(.+)\s*/$1/ for (@lines);
 
@@ -1027,6 +1032,28 @@ sub vcs_find_signers {
     return ($commits, @lines);
 }
 
+sub vcs_find_author {
+    my ($cmd) = @_;
+    my @lines = ();
+
+    @lines = &{$VCS_cmds{"execute_cmd"}}($cmd);
+
+    if (!$email_git_penguin_chiefs) {
+       @lines = grep(!/${penguin_chiefs}/i, @lines);
+    }
+
+    return @lines if !@lines;
+
+## Reformat email addresses (with names) to avoid badly written signatures
+
+    foreach my $line (@lines) {
+       my ($name, $address) = parse_email($line);
+       $line = format_email($name, $address, 1);
+    }
+
+    return @lines;
+}
+
 sub vcs_save_commits {
     my ($cmd) = @_;
     my @lines = ();
@@ -1084,6 +1111,10 @@ sub vcs_blame {
        @commits = vcs_save_commits($cmd);
     }
 
+    foreach my $commit (@commits) {
+       $commit =~ s/^\^//g;
+    }
+
     return @commits;
 }
 
@@ -1121,6 +1152,8 @@ sub vcs_assign {
        @lines = mailmap(@lines);
     }
 
+    return if (@lines <= 0);
+
     @lines = sort(@lines);
 
     # uniq -c
@@ -1165,14 +1198,17 @@ sub vcs_file_blame {
     my ($file) = @_;
 
     my @signers = ();
+    my @all_commits = ();
     my @commits = ();
     my $total_commits;
+    my $total_lines;
 
     return if (!vcs_exists());
 
-    @commits = vcs_blame($file);
-    @commits = uniq(@commits);
+    @all_commits = vcs_blame($file);
+    @commits = uniq(@all_commits);
     $total_commits = @commits;
+    $total_lines = @all_commits;
 
     foreach my $commit (@commits) {
        my $commit_count;
@@ -1182,10 +1218,28 @@ sub vcs_file_blame {
        $cmd =~ s/(\$\w+)/$1/eeg;       #interpolate $cmd
 
        ($commit_count, @commit_signers) = vcs_find_signers($cmd);
+
        push(@signers, @commit_signers);
     }
 
     if ($from_filename) {
+       if ($output_rolestats) {
+           my @blame_signers;
+           foreach my $commit (@commits) {
+               my $i;
+               my $cmd = $VCS_cmds{"find_commit_author_cmd"};
+               $cmd =~ s/(\$\w+)/$1/eeg;       #interpolate $cmd
+               my @author = vcs_find_author($cmd);
+               next if !@author;
+               my $count = grep(/$commit/, @all_commits);
+               for ($i = 0; $i < $count ; $i++) {
+                   push(@blame_signers, $author[0]);
+               }
+           }
+           if (@blame_signers) {
+               vcs_assign("authored lines", $total_lines, @blame_signers);
+           }
+       }
        vcs_assign("commits", $total_commits, @signers);
     } else {
        vcs_assign("modified commits", $total_commits, @signers);