* [PATCH] MAINTAINERS: xfs@oss.sgi.com is a list
@ 2013-07-02 15:00 Yann Droneaud
2013-07-03 6:40 ` Dave Chinner
0 siblings, 1 reply; 8+ messages in thread
From: Yann Droneaud @ 2013-07-02 15:00 UTC (permalink / raw)
To: linux-kernel, xfs; +Cc: Yann Droneaud
This patch changes type of xfs@oss.sgi.com
The output of ./scripts/get_maintainer.pl is
modified to report xfs@oss.sgi.com as a list:
Ben Myers <bpm@sgi.com> (supporter:XFS FILESYSTEM)
Alex Elder <elder@kernel.org> (supporter:XFS FILESYSTEM)
-xfs@oss.sgi.com (supporter:XFS FILESYSTEM)
+xfs@oss.sgi.com (open list:XFS FILESYSTEM)
linux-kernel@vger.kernel.org (open list)
Signed-off-by: Yann Droneaud <ydroneaud@opteya.com>
---
MAINTAINERS | 1 -
1 file changed, 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index ad7e322..7b6ab51 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -9139,7 +9139,6 @@ XFS FILESYSTEM
P: Silicon Graphics Inc
M: Ben Myers <bpm@sgi.com>
M: Alex Elder <elder@kernel.org>
-M: xfs@oss.sgi.com
L: xfs@oss.sgi.com
W: http://oss.sgi.com/projects/xfs
T: git git://oss.sgi.com/xfs/xfs.git
--
1.8.3.1
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] MAINTAINERS: xfs@oss.sgi.com is a list
2013-07-02 15:00 [PATCH] MAINTAINERS: xfs@oss.sgi.com is a list Yann Droneaud
@ 2013-07-03 6:40 ` Dave Chinner
2013-07-03 8:14 ` Yann Droneaud
0 siblings, 1 reply; 8+ messages in thread
From: Dave Chinner @ 2013-07-03 6:40 UTC (permalink / raw)
To: Yann Droneaud; +Cc: linux-kernel, xfs
On Tue, Jul 02, 2013 at 05:00:47PM +0200, Yann Droneaud wrote:
> This patch changes type of xfs@oss.sgi.com
>
> The output of ./scripts/get_maintainer.pl is
> modified to report xfs@oss.sgi.com as a list:
What's the problem with that? All XFS patches and problem
reports should be sent to the xfs@oss.sgi.com list. There are far
more people than just the maintainer that can triage problems,
answer questions and review patches...
Cheers,
Dave.
--
Dave Chinner
david@fromorbit.com
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MAINTAINERS: xfs@oss.sgi.com is a list
2013-07-03 6:40 ` Dave Chinner
@ 2013-07-03 8:14 ` Yann Droneaud
2013-07-03 9:24 ` Dave Chinner
0 siblings, 1 reply; 8+ messages in thread
From: Yann Droneaud @ 2013-07-03 8:14 UTC (permalink / raw)
To: Dave Chinner; +Cc: ydroneaud, linux-kernel, xfs
Hi,
Le 03.07.2013 08:40, Dave Chinner a écrit :
> On Tue, Jul 02, 2013 at 05:00:47PM +0200, Yann Droneaud wrote:
>> This patch changes type of xfs@oss.sgi.com
>>
>> The output of ./scripts/get_maintainer.pl is
>> modified to report xfs@oss.sgi.com as a list:
>
> What's the problem with that? All XFS patches and problem
> reports should be sent to the xfs@oss.sgi.com list. There are far
> more people than just the maintainer that can triage problems,
> answer questions and review patches...
>
It was just disturbing: I was looking for a list for XFS
and found only maintainers.
Except dm-devel, no others public mailing list is used as a maintainer.
So I thought it was a misinterpretation after following commit:
commit 18caa67ad41212e6f82a675c40f461ffb45f098e
Author: Ben Myers <bpm@sgi.com>
Date: Thu Apr 12 17:05:05 2012 +0000
MAINTAINERS: retire xfs-masters@oss.sgi.com
xfs-masters@oss.sgi.com will be retired in favor of xfs@oss.sgi.com
sometime soon.
Signed-off-by: Ben Myers <bpm@sgi.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
diff --git a/MAINTAINERS b/MAINTAINERS
index 2dcfca8..12b0445 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -7564,7 +7564,7 @@ XFS FILESYSTEM
P: Silicon Graphics Inc
M: Ben Myers <bpm@sgi.com>
M: Alex Elder <elder@kernel.org>
-M: xfs-masters@oss.sgi.com
+M: xfs@oss.sgi.com
L: xfs@oss.sgi.com
W: http://oss.sgi.com/projects/xfs
T: git git://oss.sgi.com/xfs/xfs.git
Regards.
--
Yann Droneaud
OPTEYA
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH] MAINTAINERS: xfs@oss.sgi.com is a list
2013-07-03 8:14 ` Yann Droneaud
@ 2013-07-03 9:24 ` Dave Chinner
2013-07-03 9:36 ` Yann Droneaud
0 siblings, 1 reply; 8+ messages in thread
From: Dave Chinner @ 2013-07-03 9:24 UTC (permalink / raw)
To: Yann Droneaud; +Cc: linux-kernel, xfs
On Wed, Jul 03, 2013 at 10:14:41AM +0200, Yann Droneaud wrote:
> Hi,
>
> Le 03.07.2013 08:40, Dave Chinner a écrit :
> >On Tue, Jul 02, 2013 at 05:00:47PM +0200, Yann Droneaud wrote:
> >>This patch changes type of xfs@oss.sgi.com
> >>
> >>The output of ./scripts/get_maintainer.pl is
> >>modified to report xfs@oss.sgi.com as a list:
> >
> >What's the problem with that? All XFS patches and problem
> >reports should be sent to the xfs@oss.sgi.com list. There are far
> >more people than just the maintainer that can triage problems,
> >answer questions and review patches...
> >
>
> It was just disturbing: I was looking for a list for XFS
> and found only maintainers.
That's what the:
L: xfs@oss.sgi.com
entry is, yes?
Cheers,
Dave.
--
Dave Chinner
david@fromorbit.com
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MAINTAINERS: xfs@oss.sgi.com is a list
2013-07-03 9:24 ` Dave Chinner
@ 2013-07-03 9:36 ` Yann Droneaud
2013-07-03 9:50 ` Dave Chinner
2013-07-03 9:54 ` Joe Perches
0 siblings, 2 replies; 8+ messages in thread
From: Yann Droneaud @ 2013-07-03 9:36 UTC (permalink / raw)
To: Dave Chinner; +Cc: ydroneaud, linux-kernel, xfs
Le 03.07.2013 11:24, Dave Chinner a écrit :
> On Wed, Jul 03, 2013 at 10:14:41AM +0200, Yann Droneaud wrote:
>> Le 03.07.2013 08:40, Dave Chinner a écrit :
>> >On Tue, Jul 02, 2013 at 05:00:47PM +0200, Yann Droneaud wrote:
>> >>This patch changes type of xfs@oss.sgi.com
>> >>
>> >>The output of ./scripts/get_maintainer.pl is
>> >>modified to report xfs@oss.sgi.com as a list:
>> >
>> >What's the problem with that? All XFS patches and problem
>> >reports should be sent to the xfs@oss.sgi.com list. There are far
>> >more people than just the maintainer that can triage problems,
>> >answer questions and review patches...
>> >
>>
>> It was just disturbing: I was looking for a list for XFS
>> and found only maintainers.
>
> That's what the:
>
> L: xfs@oss.sgi.com
>
> entry is, yes?
>
In the output of ./scripts/get_maintainer.pl:
Ben Myers <bpm@sgi.com> (supporter:XFS FILESYSTEM)
Alex Elder <elder@kernel.org> (supporter:XFS FILESYSTEM)
xfs@oss.sgi.com (supporter:XFS FILESYSTEM)
linux-kernel@vger.kernel.org (open list)
[If having the mailing list listed as the maintainer/supporter is
correct,
could ./scripts/get_maintainer.pl also report the mailing list as a
list ?]
Regards.
--
Yann Droneaud
OPTEYA
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MAINTAINERS: xfs@oss.sgi.com is a list
2013-07-03 9:36 ` Yann Droneaud
@ 2013-07-03 9:50 ` Dave Chinner
2013-07-03 9:54 ` Joe Perches
1 sibling, 0 replies; 8+ messages in thread
From: Dave Chinner @ 2013-07-03 9:50 UTC (permalink / raw)
To: Yann Droneaud; +Cc: linux-kernel, xfs
On Wed, Jul 03, 2013 at 11:36:39AM +0200, Yann Droneaud wrote:
> Le 03.07.2013 11:24, Dave Chinner a écrit :
> >On Wed, Jul 03, 2013 at 10:14:41AM +0200, Yann Droneaud wrote:
> >>Le 03.07.2013 08:40, Dave Chinner a écrit :
> >>>On Tue, Jul 02, 2013 at 05:00:47PM +0200, Yann Droneaud wrote:
> >>>>This patch changes type of xfs@oss.sgi.com
> >>>>
> >>>>The output of ./scripts/get_maintainer.pl is
> >>>>modified to report xfs@oss.sgi.com as a list:
> >>>
> >>>What's the problem with that? All XFS patches and problem
> >>>reports should be sent to the xfs@oss.sgi.com list. There are far
> >>>more people than just the maintainer that can triage problems,
> >>>answer questions and review patches...
> >>>
> >>
> >>It was just disturbing: I was looking for a list for XFS
> >>and found only maintainers.
> >
> >That's what the:
> >
> >L: xfs@oss.sgi.com
> >
> >entry is, yes?
> >
>
> In the output of ./scripts/get_maintainer.pl:
>
> Ben Myers <bpm@sgi.com> (supporter:XFS FILESYSTEM)
> Alex Elder <elder@kernel.org> (supporter:XFS FILESYSTEM)
> xfs@oss.sgi.com (supporter:XFS FILESYSTEM)
> linux-kernel@vger.kernel.org (open list)
You're smarter than a dumb script. If the information that the
script parses is correct and the dumb script doesn't give you the
right information, then what needs fixing?
Seriously, all you are proving is the old adage that
scripts/get_maintainer.pl should be considered harmful because
people use it without first engaging their brain.
Cheers,
Dave.
--
Dave Chinner
david@fromorbit.com
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MAINTAINERS: xfs@oss.sgi.com is a list
2013-07-03 9:36 ` Yann Droneaud
2013-07-03 9:50 ` Dave Chinner
@ 2013-07-03 9:54 ` Joe Perches
1 sibling, 0 replies; 8+ messages in thread
From: Joe Perches @ 2013-07-03 9:54 UTC (permalink / raw)
To: Yann Droneaud; +Cc: linux-kernel, xfs
On Wed, 2013-07-03 at 11:36 +0200, Yann Droneaud wrote:
> In the output of ./scripts/get_maintainer.pl:
>
> Ben Myers <bpm@sgi.com> (supporter:XFS FILESYSTEM)
> Alex Elder <elder@kernel.org> (supporter:XFS FILESYSTEM)
> xfs@oss.sgi.com (supporter:XFS FILESYSTEM)
> linux-kernel@vger.kernel.org (open list)
>
> [If having the mailing list listed as the maintainer/supporter is
> correct,
> could ./scripts/get_maintainer.pl also report the mailing list as a
> list ?]
It does when you ask for just the lists...
$ ./scripts/get_maintainer.pl -f -l -nom fs/xfs
xfs@oss.sgi.com (open list:XFS FILESYSTEM)
linux-kernel@vger.kernel.org (open list)
If it bothers you that the same email address
doesn't get reported as both a maintainer and
a mailing list, you're free to submit a patch
to get_maintainer.pl. (look at sub add_role)
It doesn't bother me.
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MAINTAINERS: xfs@oss.sgi.com is a list
@ 2013-07-03 10:07 Yann Droneaud
0 siblings, 0 replies; 8+ messages in thread
From: Yann Droneaud @ 2013-07-03 10:07 UTC (permalink / raw)
To: Dave Chinner; +Cc: ydroneaud, linux-kernel, xfs
Le 03.07.2013 11:50, Dave Chinner a écrit :
> On Wed, Jul 03, 2013 at 11:36:39AM +0200, Yann Droneaud wrote:
>> Le 03.07.2013 11:24, Dave Chinner a écrit :
>> >On Wed, Jul 03, 2013 at 10:14:41AM +0200, Yann Droneaud wrote:
>> >>Le 03.07.2013 08:40, Dave Chinner a écrit :
>> >>>On Tue, Jul 02, 2013 at 05:00:47PM +0200, Yann Droneaud wrote:
>> >>>>This patch changes type of xfs@oss.sgi.com
>> >>>>
>> >>>>The output of ./scripts/get_maintainer.pl is
>> >>>>modified to report xfs@oss.sgi.com as a list:
>> >>>
>> >>>What's the problem with that? All XFS patches and problem
>> >>>reports should be sent to the xfs@oss.sgi.com list. There are far
>> >>>more people than just the maintainer that can triage problems,
>> >>>answer questions and review patches...
>> >>>
>> >>
>> >>It was just disturbing: I was looking for a list for XFS
>> >>and found only maintainers.
>> >
>> >That's what the:
>> >
>> >L: xfs@oss.sgi.com
>> >
>> >entry is, yes?
>> >
>>
>> In the output of ./scripts/get_maintainer.pl:
>>
>> Ben Myers <bpm@sgi.com> (supporter:XFS FILESYSTEM)
>> Alex Elder <elder@kernel.org> (supporter:XFS FILESYSTEM)
>> xfs@oss.sgi.com (supporter:XFS FILESYSTEM)
>> linux-kernel@vger.kernel.org (open list)
>
> You're smarter than a dumb script. If the information that the
> script parses is correct and the dumb script doesn't give you the
> right information, then what needs fixing?
>
And what about:
If you are the only one[*] to abuse ./scripts/get_maintainer.pl,
then what needs fixing ?
[*] I have found three other examples of duplicating mailing list as
maintainer:
$ sed -n 's/^L:[[:space:]]*//p' MAINTAINERS | sort | uniq > L
$ sed -n 's/^M:[[:space:]]*//p' MAINTAINERS | sort | uniq > M
$ comm -12 L M
ceph-devel@vger.kernel.org
dm-devel@redhat.com
xfs@oss.sgi.com
I've sent a patch for xfs@oss.sgi.com and dm-devel@redhat.com,
in my first pass I've missed ceph-devel@vger.kernel.org
> Seriously, all you are proving is the old adage that
> scripts/get_maintainer.pl should be considered harmful because
> people use it without first engaging their brain.
Just thinking about the whole picture.
In this case ./scripts/get_maintainer.pl is going to be right most of
time
except for 3 subsystems on about 1160. So having the same address for
maintainer
and list is a rather uncommon case. Which could be misleading for most.
Regards.
--
Yann Droneaud
OPTEYA
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2013-07-03 10:07 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-07-02 15:00 [PATCH] MAINTAINERS: xfs@oss.sgi.com is a list Yann Droneaud
2013-07-03 6:40 ` Dave Chinner
2013-07-03 8:14 ` Yann Droneaud
2013-07-03 9:24 ` Dave Chinner
2013-07-03 9:36 ` Yann Droneaud
2013-07-03 9:50 ` Dave Chinner
2013-07-03 9:54 ` Joe Perches
-- strict thread matches above, loose matches on Subject: below --
2013-07-03 10:07 Yann Droneaud
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox