Messages in this thread | | | Subject | Re: [PATCH] get_maintainer.pl: Add support for moderated lists | From | Joe Perches <> | Date | Wed, 29 Feb 2012 11:58:46 -0800 |
| |
On Wed, 2012-02-29 at 20:49 +0100, Richard Weinberger wrote: > Currently get_maintainer.pl reports moderated lists as open, > which is just wrong.
Hi Richard.
<shrug> From my perspective, it's correct and simple.
Moderated lists just means your email may get delayed and you may get a "delayed due to moderation" email.
It's not a big deal to me though if Andrew wants to pick this up. It seems that all current lists are marked the same way. You might consider using a case insensitive m/\(moderated/i though in case some list has moderated in the email address itself.
cheers, Joe
> Signed-off-by: Richard Weinberger <richard@nod.at> > --- > scripts/get_maintainer.pl | 9 +++++++-- > 1 files changed, 7 insertions(+), 2 deletions(-) > > diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl > index f32a04c..b9bcfb6 100755 > --- a/scripts/get_maintainer.pl > +++ b/scripts/get_maintainer.pl > @@ -1027,8 +1027,13 @@ sub add_categories { > if ($email_list) { > if (!$hash_list_to{lc($list_address)}) { > $hash_list_to{lc($list_address)} = 1; > - push(@list_to, [$list_address, > - "open list${list_role}"]); > + if ($list_additional =~ m/moderated/) { > + push(@list_to, [$list_address, > + "moderated list${list_role}"]); > + } else { > + push(@list_to, [$list_address, > + "open list${list_role}"]); > + } > } > } > }
| |