[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [xen master] get_maintainer: Add THE REST for sections with reviewers only
commit 7114bbfc8424ee467c6c8c82f077764ca4fa799b Author: Anthony PERARD <anthony.perard@xxxxxxxxxx> AuthorDate: Tue Oct 17 09:53:34 2023 +0200 Commit: Jan Beulich <jbeulich@xxxxxxxx> CommitDate: Tue Oct 17 09:53:34 2023 +0200 get_maintainer: Add THE REST for sections with reviewers only Sometime, a contributer would like to be CCed on part of the changes, and it could happen that we end-up with a section that doesn't have any maintainer, but a Ack from a maintainer would still be needed. Rework get_maintainer so if there's no maintainers beside THE REST, it doesn't drop THE REST emails. Signed-off-by: Anthony PERARD <anthony.perard@xxxxxxxxxx> Reviewed-by: Julien Grall <jgrall@xxxxxxxxxx> Release-acked-by: Henry Wang <Henry.Wang@xxxxxxx> --- scripts/get_maintainer.pl | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl index cf629cdf3c..533d0df72a 100755 --- a/scripts/get_maintainer.pl +++ b/scripts/get_maintainer.pl @@ -732,8 +732,15 @@ sub get_maintainers { my @email_new; my $do_replace = 0; foreach my $email (@email_to) { - if ($email->[1] ne 'supporter:THE REST') { + # Replace @email_to list with a list which drop "THE REST" if + # there's a role other than "reviewer", that is if there's a + # maintainer/supporter in a section other than THE REST. + if ($email->[1] ne 'supporter:THE REST' and $email->[1] ne 'reviewer') { $do_replace = 1; + } + # Prepare a new list without "THE REST", to be used if $do_replace + # is true. + if ($email->[1] ne 'supporter:THE REST') { push @email_new, $email; } } -- generated by git-patchbot for /home/xen/git/xen.git#master
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |