Re: [PATCH] MAINTAINERS: rectify file patterns for NETFILTER

From: Pablo Neira Ayuso
Date: Mon Nov 16 2020 - 04:43:17 EST


Hi Lukas,

On Sun, Nov 15, 2020 at 07:58:33PM -0800, Joe Perches wrote:
> On Mon, 2020-11-09 at 10:19 +0100, Lukas Bulwahn wrote:
> > The two file patterns in the NETFILTER section:
> >
> >   F: include/linux/netfilter*
> >   F: include/uapi/linux/netfilter*
> >
> > intended to match the directories:
> >
> >   ./include{/uapi}/linux/netfilter_{arp,bridge,ipv4,ipv6}
> >
> > A quick check with ./scripts/get_maintainer.pl --letters -f will show that
> > they are not matched, though, because this pattern only matches files, but
> > not directories.
> >
> > Rectify the patterns to match the intended directories.
> []
> diff --git a/MAINTAINERS b/MAINTAINERS
> []
> > @@ -12139,10 +12139,10 @@ W: http://www.nftables.org/
> >  Q: http://patchwork.ozlabs.org/project/netfilter-devel/list/
> >  T: git git://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf.git
> >  T: git git://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf-next.git
> > -F: include/linux/netfilter*
> > +F: include/linux/netfilter*/
> >  F: include/linux/netfilter/
>
> This line could be deleted or perhaps moved up one line above
>
> F: include/linux/netfilter/
> F: include/linux/netfilter*/
>
> (as the second line already matches the first line's files too)
>
> >  F: include/net/netfilter/
> > -F: include/uapi/linux/netfilter*
> > +F: include/uapi/linux/netfilter*/
> >  F: include/uapi/linux/netfilter/
>
> same here.
>
> >  F: net/*/netfilter.c
> >  F: net/*/netfilter/

Please, send a v2 to address this feedback. Thank you.