[PATCH] [70/2many] MAINTAINERS - ARPD SUPPORT

[PATCH] [70/2many] MAINTAINERS - ARPD SUPPORT

[joe]

Add file pattern to MAINTAINER entry

Signed-off-by: Joe Perches <joe@perches.com>

diff --git a/MAINTAINERS b/MAINTAINERS
index 90c1b81..ac2226b 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -697,6 +697,7 @@ ARPD SUPPORT
 P:	Jonathan Layes
 L:	netdev@vger.kernel.org
 S:	Maintained
+F:	net/ipv4/arp.c
 
 ASUS ACPI EXTRAS DRIVER
 P:	Corentin Chary

Re: [PATCH] [70/2many] MAINTAINERS - ARPD SUPPORT

[Alan Cox]

On Sun, 12 Aug 2007 23:23:36 -0700
joe@perches.com wrote:

> Add file pattern to MAINTAINER entry
> 
> Signed-off-by: Joe Perches <joe@perches.com>
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 90c1b81..ac2226b 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -697,6 +697,7 @@ ARPD SUPPORT
>  P:	Jonathan Layes
>  L:	netdev@vger.kernel.org
>  S:	Maintained
> +F:	net/ipv4/arp.c

NAK

arp.c is the netdev people, ARPD is a corner case bit of code within it,
something your patterns don't actually cope with

Re: [PATCH] [70/2many] MAINTAINERS - ARPD SUPPORT

[Joe Perches]

On Mon, 2007-08-13 at 11:49 +0100, Alan Cox wrote:
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 90c1b81..ac2226b 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -697,6 +697,7 @@ ARPD SUPPORT
> >  P:	Jonathan Layes
> >  L:	netdev@vger.kernel.org
> >  S:	Maintained
> > +F:	net/ipv4/arp.c
> 
> NAK
> 
> arp.c is the netdev people, ARPD is a corner case bit of code within it,
> something your patterns don't actually cope with

Suggestions?

Re: [PATCH] [70/2many] MAINTAINERS - ARPD SUPPORT

[Alan Cox]

On Mon, 13 Aug 2007 08:46:06 -0700
Joe Perches <joe@perches.com> wrote:

> On Mon, 2007-08-13 at 11:49 +0100, Alan Cox wrote:
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index 90c1b81..ac2226b 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -697,6 +697,7 @@ ARPD SUPPORT
> > >  P:	Jonathan Layes
> > >  L:	netdev@vger.kernel.org
> > >  S:	Maintained
> > > +F:	net/ipv4/arp.c
> > 
> > NAK
> > 
> > arp.c is the netdev people, ARPD is a corner case bit of code within it,
> > something your patterns don't actually cope with
> 
> Suggestions?

I wouldn't add a pattern for this. Actually I think the entire thing is a
bad idea but thats another matter.

Re: [PATCH] [70/2many] MAINTAINERS - ARPD SUPPORT

[Joe Perches]

On Mon, 2007-08-13 at 17:35 +0100, Alan Cox wrote:
> I wouldn't add a pattern for this.

Back to:ARPD SUPPORT
P:	Jonathan Layes
L:	netdev@vger.kernel.org
S:	Maintained

> Actually I think the entire thing is a
> bad idea but thats another matter.

Of course it's not an end-all solution,
but what might work better?

Re: [PATCH] [70/2many] MAINTAINERS - ARPD SUPPORT

[Alan Cox]

On Mon, 13 Aug 2007 10:04:19 -0700
Joe Perches <joe@perches.com> wrote:

> On Mon, 2007-08-13 at 17:35 +0100, Alan Cox wrote:
> > I wouldn't add a pattern for this.
> 
> Back to:ARPD SUPPORT
> P:	Jonathan Layes
> L:	netdev@vger.kernel.org
> S:	Maintained
> 
> > Actually I think the entire thing is a
> > bad idea but thats another matter.
> 
> Of course it's not an end-all solution,
> but what might work better?

Working off the git tree as it shows who actually is making
changes/updating stuff recently and why which is a major clue when
tracing bugs

Re: [PATCH] [70/2many] MAINTAINERS - ARPD SUPPORT

[Stefan Richter]

Alan Cox wrote:
>> > Actually I think the entire thing is a
>> > bad idea but thats another matter.

> Working off the git tree as it shows who actually is making
> changes/updating stuff recently and why which is a major clue when
> tracing bugs

Adrian Bunk wrote in http://lkml.org/lkml/2007/6/30/107 :
| I don't see how git could tell you to Cc net patches to
| netdev@vger.kernel.org.

Ditto with problem reports.

For example, Cc'ing linux1394-devel on FireWire topics is a lot more
important than Cc'ing me.
-- 
Stefan Richter
-=====-=-=== =--- -====
http://arcgraph.de/sr/

Re: [PATCH] [70/2many] MAINTAINERS - ARPD SUPPORT

[Alan Cox]

On Wed, 15 Aug 2007 10:10:38 +0200
Stefan Richter <stefanr@s5r6.in-berlin.de> wrote:

> Alan Cox wrote:
> >> > Actually I think the entire thing is a
> >> > bad idea but thats another matter.
> 
> > Working off the git tree as it shows who actually is making
> > changes/updating stuff recently and why which is a major clue when
> > tracing bugs
> 
> Adrian Bunk wrote in http://lkml.org/lkml/2007/6/30/107 :
> | I don't see how git could tell you to Cc net patches to
> | netdev@vger.kernel.org.
> 
> Ditto with problem reports.
> 
> For example, Cc'ing linux1394-devel on FireWire topics is a lot more
> important than Cc'ing me.

Tools question. And even if you need to keep a simple set of mappings by
hand for mailing list/files thats a fairly small set of patterns