[XEN PATCH] get_maintainer: Add THE REST for sections with reviewers only

[Anthony PERARD <anthony.perard@xxxxxxxxxx>]

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>
---

Notes:
    Soon, the RISCV section might be the first (I think) section without
    maintainer and without been nested in another section other than THE
    REST.
    
    Tests by checking that the following comnands returns the expected emails.
    ./scripts/get_maintainer.pl -f tools/libs/Makefile
    ./scripts/get_maintainer.pl -f xen/arch/riscv/Makefile
    ./scripts/get_maintainer.pl -f xen/arch/arm/Makefile
    
    An extra '--sections' argument list all sections, including THE REST.

 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;
             }
         }
-- 
Anthony PERARD