get_maintainer.pl: Adjust to Xen workflow

Based on feedback from reviewers:
* Disable git fallback by default: it has a tendency to mail
  anyone who did a single oneline change and should not be
  necessary for a project of Xen's size.
* Disable rolestats: Makes cut-and-paste from the output into the
  commit message easy.
* Drop "THE REST" fallback: Don't spam Keir *too* much.

Signed-off-by: Don Slutz <dslutz@verizon.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
[ijc -- expanded the changelog]
This commit is contained in:
Don Slutz authored and Ian Campbell committed 2013-11-19 14:02:23 +00:00
1 parent f85e314d26
commit d6738db02d
1 file changed
+17 -2
+17 -2
View File
@@ -28,7 +28,7 @@ my $email_git = 0;
my $email_git_all_signature_types = 0;
my $email_git_blame = 0;
my $email_git_blame_signatures = 1;
my $email_git_fallback = 1;
my $email_git_fallback = 0;
my $email_git_min_signatures = 1;
my $email_git_max_maintainers = 5;
my $email_git_min_percent = 5;
@@ -37,10 +37,11 @@ my $email_hg_since = "-365";
my $interactive = 0;
my $email_remove_duplicates = 1;
my $email_use_mailmap = 1;
my $email_drop_the_rest_supporter_if_supporter_found = 1;
my $output_multiline = 1;
my $output_separator = ", ";
my $output_roles = 0;
my $output_rolestats = 1;
my $output_rolestats = 0;
my $scm = 0;
my $web = 0;
my $subsystem = 0;
@@ -196,6 +197,7 @@ if (!GetOptions(
'i|interactive!' => \$interactive,
'remove-duplicates!' => \$email_remove_duplicates,
'mailmap!' => \$email_use_mailmap,
'drop_the_rest_supporter!' => \$email_drop_the_rest_supporter_if_supporter_found,
'm!' => \$email_maintainer,
'n!' => \$email_usename,
'l!' => \$email_list,
@@ -651,6 +653,19 @@ sub get_maintainers {
}
}
if ($email_drop_the_rest_supporter_if_supporter_found && $#email_to > 0) {
my @email_new;
my $do_replace = 0;
foreach my $email (@email_to) {
if ($email->[1] ne 'supporter:THE REST') {
$do_replace = 1;
push @email_new, $email;
}
}
@email_to = @email_new
if $do_replace;
}
foreach my $email (@email_to, @list_to) {
$email->[0] = deduplicate_email($email->[0]);
}
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.