[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


  • To: Anthony PERARD <anthony.perard@xxxxxxxxxx>
  • From: Henry Wang <Henry.Wang@xxxxxxx>
  • Date: Fri, 6 Oct 2023 14:01:59 +0000
  • Accept-language: zh-CN, en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=MFpcY2eOiSAkd1LC7JVS31B9R0FGGJUsrTImZaX50TQ=; b=FSy8mR36mN2611765yNZ6w0cW4JvLSFAUueTy9cNVeSIedNmssuOdZaUFGtnidsCQZy82OlP4MY/cJeeCMrZB2QGJFUhQyVwka5r4m7qg8an3bz51PJTSrqRd4QwYl2AYh+vjQfWPwRd5ttWnO2X8W7JWK4jZeYkzwiHiZOuX/nde0Y8TE3fGBqsfadgSUFSMmKlwhA0qHXTnd6c3lIr67x02d3Mtl7zqCRDFBtj3/qlO0JgiQp7nWW1iKGCmHGbhCMAIde+a07EfmQSV8dkP94XfWhuxU6xQ4dEm/+K1LfM7bT0KC6cTuPejlBsyKPUS5CFdFht4ASBr1mccJdKmQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=BzXaiKYmDq3SCv5miwMtFfCoORQxVOUWHaufDpnbjKsYzrjei6Pfo17inxMsfdDV5Yg70zmlpwyZ4zXZL0NUo6zq0eg25JyYm6/NnHOuKwL+HLqFkVZl5CTqc0TXoytUI7G+Gq+vAYTmHt9cqhTuKmlnzjCqew59j8obRw1DxgIY3Xaymd4YIDSkYvz/eOJvwzTefc1KwZslytRncvB20r458e5ZedmcXmaSanvWQiQDav5aUyK+g43VzlGzBj3GffNZ5BYZVi+PaYdqREJkK279WCildBp1bjeszuYsbQOR/CEUlTCubDQYZ0fSZ56gOlQt1JFzgwZ2c5EBijY8UQ==
  • Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
  • Delivery-date: Fri, 06 Oct 2023 14:02:20 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHZ+FyiaBVN8FQsE0aCzepjg4+08LA8ypOA
  • Thread-topic: [XEN PATCH] get_maintainer: Add THE REST for sections with reviewers only

Hi Anthony,

> On Oct 6, 2023, at 21:54, Anthony PERARD <anthony.perard@xxxxxxxxxx> wrote:
> 
> 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>

Release-acked-by: Henry Wang <Henry.Wang@xxxxxxx>

Kind regards,
Henry

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




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.