get_maintainer.pl: Remove the --git-chief-penguins option
Linus likely does not want to get e-mails about QEMU, so let's just remove this option. Suggested-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Thomas Huth <thuth@linux.vnet.ibm.com> Reviewed-by: Don Slutz <dslutz@verizon.com> Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
This commit is contained in:
parent
8ccefb91bf
commit
80b57dda89
@ -23,7 +23,6 @@ my $email_usename = 1;
|
|||||||
my $email_maintainer = 1;
|
my $email_maintainer = 1;
|
||||||
my $email_list = 1;
|
my $email_list = 1;
|
||||||
my $email_subscriber_list = 0;
|
my $email_subscriber_list = 0;
|
||||||
my $email_git_penguin_chiefs = 0;
|
|
||||||
my $email_git = 0;
|
my $email_git = 0;
|
||||||
my $email_git_all_signature_types = 0;
|
my $email_git_all_signature_types = 0;
|
||||||
my $email_git_blame = 0;
|
my $email_git_blame = 0;
|
||||||
@ -60,21 +59,6 @@ my $exit = 0;
|
|||||||
my %commit_author_hash;
|
my %commit_author_hash;
|
||||||
my %commit_signer_hash;
|
my %commit_signer_hash;
|
||||||
|
|
||||||
my @penguin_chief = ();
|
|
||||||
push(@penguin_chief, "Linus Torvalds:torvalds\@linux-foundation.org");
|
|
||||||
#Andrew wants in on most everything - 2009/01/14
|
|
||||||
#push(@penguin_chief, "Andrew Morton:akpm\@linux-foundation.org");
|
|
||||||
|
|
||||||
my @penguin_chief_names = ();
|
|
||||||
foreach my $chief (@penguin_chief) {
|
|
||||||
if ($chief =~ m/^(.*):(.*)/) {
|
|
||||||
my $chief_name = $1;
|
|
||||||
my $chief_addr = $2;
|
|
||||||
push(@penguin_chief_names, $chief_name);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
my $penguin_chiefs = "\(" . join("|", @penguin_chief_names) . "\)";
|
|
||||||
|
|
||||||
# Signature types of people who are either
|
# Signature types of people who are either
|
||||||
# a) responsible for the code in question, or
|
# a) responsible for the code in question, or
|
||||||
# b) familiar enough with it to give relevant feedback
|
# b) familiar enough with it to give relevant feedback
|
||||||
@ -187,7 +171,6 @@ if (!GetOptions(
|
|||||||
'git-blame!' => \$email_git_blame,
|
'git-blame!' => \$email_git_blame,
|
||||||
'git-blame-signatures!' => \$email_git_blame_signatures,
|
'git-blame-signatures!' => \$email_git_blame_signatures,
|
||||||
'git-fallback!' => \$email_git_fallback,
|
'git-fallback!' => \$email_git_fallback,
|
||||||
'git-chief-penguins!' => \$email_git_penguin_chiefs,
|
|
||||||
'git-min-signatures=i' => \$email_git_min_signatures,
|
'git-min-signatures=i' => \$email_git_min_signatures,
|
||||||
'git-max-maintainers=i' => \$email_git_max_maintainers,
|
'git-max-maintainers=i' => \$email_git_max_maintainers,
|
||||||
'git-min-percent=i' => \$email_git_min_percent,
|
'git-min-percent=i' => \$email_git_min_percent,
|
||||||
@ -256,7 +239,7 @@ if ($sections) {
|
|||||||
|
|
||||||
if ($email &&
|
if ($email &&
|
||||||
($email_maintainer + $email_list + $email_subscriber_list +
|
($email_maintainer + $email_list + $email_subscriber_list +
|
||||||
$email_git + $email_git_penguin_chiefs + $email_git_blame) == 0) {
|
$email_git + $email_git_blame) == 0) {
|
||||||
die "$P: Please select at least 1 email option\n";
|
die "$P: Please select at least 1 email option\n";
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -671,19 +654,6 @@ sub get_maintainers {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
foreach my $chief (@penguin_chief) {
|
|
||||||
if ($chief =~ m/^(.*):(.*)/) {
|
|
||||||
my $email_address;
|
|
||||||
|
|
||||||
$email_address = format_email($1, $2, $email_usename);
|
|
||||||
if ($email_git_penguin_chiefs) {
|
|
||||||
push(@email_to, [$email_address, 'chief penguin']);
|
|
||||||
} else {
|
|
||||||
@email_to = grep($_->[0] !~ /${email_address}/, @email_to);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
foreach my $email (@file_emails) {
|
foreach my $email (@file_emails) {
|
||||||
my ($name, $address) = parse_email($email);
|
my ($name, $address) = parse_email($email);
|
||||||
|
|
||||||
@ -740,7 +710,6 @@ MAINTAINER field selection options:
|
|||||||
--git-all-signature-types => include signers regardless of signature type
|
--git-all-signature-types => include signers regardless of signature type
|
||||||
or use only ${signature_pattern} signers (default: $email_git_all_signature_types)
|
or use only ${signature_pattern} signers (default: $email_git_all_signature_types)
|
||||||
--git-fallback => use git when no exact MAINTAINERS pattern (default: $email_git_fallback)
|
--git-fallback => use git when no exact MAINTAINERS pattern (default: $email_git_fallback)
|
||||||
--git-chief-penguins => include ${penguin_chiefs}
|
|
||||||
--git-min-signatures => number of signatures required (default: $email_git_min_signatures)
|
--git-min-signatures => number of signatures required (default: $email_git_min_signatures)
|
||||||
--git-max-maintainers => maximum maintainers to add (default: $email_git_max_maintainers)
|
--git-max-maintainers => maximum maintainers to add (default: $email_git_max_maintainers)
|
||||||
--git-min-percent => minimum percentage of commits required (default: $email_git_min_percent)
|
--git-min-percent => minimum percentage of commits required (default: $email_git_min_percent)
|
||||||
@ -1281,10 +1250,6 @@ sub vcs_find_signers {
|
|||||||
save_commits_by_author(@lines) if ($interactive);
|
save_commits_by_author(@lines) if ($interactive);
|
||||||
save_commits_by_signer(@lines) if ($interactive);
|
save_commits_by_signer(@lines) if ($interactive);
|
||||||
|
|
||||||
if (!$email_git_penguin_chiefs) {
|
|
||||||
@signatures = grep(!/${penguin_chiefs}/i, @signatures);
|
|
||||||
}
|
|
||||||
|
|
||||||
my ($types_ref, $signers_ref) = extract_formatted_signatures(@signatures);
|
my ($types_ref, $signers_ref) = extract_formatted_signatures(@signatures);
|
||||||
|
|
||||||
return ($commits, @$signers_ref);
|
return ($commits, @$signers_ref);
|
||||||
@ -1296,10 +1261,6 @@ sub vcs_find_author {
|
|||||||
|
|
||||||
@lines = &{$VCS_cmds{"execute_cmd"}}($cmd);
|
@lines = &{$VCS_cmds{"execute_cmd"}}($cmd);
|
||||||
|
|
||||||
if (!$email_git_penguin_chiefs) {
|
|
||||||
@lines = grep(!/${penguin_chiefs}/i, @lines);
|
|
||||||
}
|
|
||||||
|
|
||||||
return @lines if !@lines;
|
return @lines if !@lines;
|
||||||
|
|
||||||
my @authors = ();
|
my @authors = ();
|
||||||
@ -1925,10 +1886,6 @@ sub vcs_file_blame {
|
|||||||
|
|
||||||
@lines = &{$VCS_cmds{"execute_cmd"}}($cmd);
|
@lines = &{$VCS_cmds{"execute_cmd"}}($cmd);
|
||||||
|
|
||||||
if (!$email_git_penguin_chiefs) {
|
|
||||||
@lines = grep(!/${penguin_chiefs}/i, @lines);
|
|
||||||
}
|
|
||||||
|
|
||||||
last if !@lines;
|
last if !@lines;
|
||||||
|
|
||||||
my @authors = ();
|
my @authors = ();
|
||||||
|
Loading…
Reference in New Issue
Block a user