* [PATCH 1/4] MAINTAINERS: Add "B:" for URI where to file bugs
@ 2016-10-20 12:22 Jani Nikula
2016-10-20 12:22 ` [PATCH 2/4] MAINTAINERS: add drm and drm/i915 bug filing info Jani Nikula
` (5 more replies)
0 siblings, 6 replies; 10+ messages in thread
From: Jani Nikula @ 2016-10-20 12:22 UTC (permalink / raw)
To: linux-kernel
Cc: intel-gfx, dri-devel, daniel, airlied, joe, akpm, Jani Nikula
Different subsystems and drivers have different preferences for where to
file bugs and what information to include. Add "B:" entry for specifying
the URI for the bug tracker directly, a web page for detailed info on
filing bugs, or a mailto: URI.
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Joe Perches <joe@perches.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
Dust settled since the last time, maybe we can make URI work?
---
MAINTAINERS | 2 ++
1 file changed, 2 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 89112a65b831..4a47ec00a09d 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -74,6 +74,8 @@ Descriptions of section entries:
These reviewers should be CCed on patches.
L: Mailing list that is relevant to this area
W: Web-page with status/info
+ B: URI for where to file bugs. A web-page with detailed bug
+ filing info, a direct bug tracker link, or a mailto: URI.
Q: Patchwork web based patch tracking system site
T: SCM tree type and location.
Type is one of: git, hg, quilt, stgit, topgit
--
2.1.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 2/4] MAINTAINERS: add drm and drm/i915 bug filing info
2016-10-20 12:22 [PATCH 1/4] MAINTAINERS: Add "B:" for URI where to file bugs Jani Nikula
@ 2016-10-20 12:22 ` Jani Nikula
2016-10-20 12:22 ` [PATCH 3/4] MAINTAINERS: Add "C:" for URI for chat where developers hang out Jani Nikula
` (4 subsequent siblings)
5 siblings, 0 replies; 10+ messages in thread
From: Jani Nikula @ 2016-10-20 12:22 UTC (permalink / raw)
To: linux-kernel
Cc: intel-gfx, dri-devel, daniel, airlied, joe, akpm, Jani Nikula
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
MAINTAINERS | 2 ++
1 file changed, 2 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 4a47ec00a09d..689f7f08a100 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3951,6 +3951,7 @@ DRM DRIVERS
M: David Airlie <airlied@linux.ie>
L: dri-devel@lists.freedesktop.org
T: git git://people.freedesktop.org/~airlied/linux
+B: https://bugs.freedesktop.org/
S: Maintained
F: drivers/gpu/drm/
F: drivers/gpu/vga/
@@ -4003,6 +4004,7 @@ M: Jani Nikula <jani.nikula@linux.intel.com>
L: intel-gfx@lists.freedesktop.org
L: dri-devel@lists.freedesktop.org
W: https://01.org/linuxgraphics/
+B: https://01.org/linuxgraphics/documentation/how-report-bugs
Q: http://patchwork.freedesktop.org/project/intel-gfx/
T: git git://anongit.freedesktop.org/drm-intel
S: Supported
--
2.1.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 3/4] MAINTAINERS: Add "C:" for URI for chat where developers hang out
2016-10-20 12:22 [PATCH 1/4] MAINTAINERS: Add "B:" for URI where to file bugs Jani Nikula
2016-10-20 12:22 ` [PATCH 2/4] MAINTAINERS: add drm and drm/i915 bug filing info Jani Nikula
@ 2016-10-20 12:22 ` Jani Nikula
2016-10-20 12:22 ` [PATCH 4/4] MAINTAINERS: add drm and drm/i915 irc channels Jani Nikula
` (3 subsequent siblings)
5 siblings, 0 replies; 10+ messages in thread
From: Jani Nikula @ 2016-10-20 12:22 UTC (permalink / raw)
To: linux-kernel
Cc: intel-gfx, dri-devel, daniel, airlied, joe, akpm, Jani Nikula
Make it easier to find the developer chat for the subsystem or driver.
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
MAINTAINERS | 2 ++
1 file changed, 2 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 689f7f08a100..53df9ccc5573 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -76,6 +76,8 @@ Descriptions of section entries:
W: Web-page with status/info
B: URI for where to file bugs. A web-page with detailed bug
filing info, a direct bug tracker link, or a mailto: URI.
+ C: URI for chat protocol, server and channel where developers
+ usually hang out, for example irc://server/channel.
Q: Patchwork web based patch tracking system site
T: SCM tree type and location.
Type is one of: git, hg, quilt, stgit, topgit
--
2.1.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 4/4] MAINTAINERS: add drm and drm/i915 irc channels
2016-10-20 12:22 [PATCH 1/4] MAINTAINERS: Add "B:" for URI where to file bugs Jani Nikula
2016-10-20 12:22 ` [PATCH 2/4] MAINTAINERS: add drm and drm/i915 bug filing info Jani Nikula
2016-10-20 12:22 ` [PATCH 3/4] MAINTAINERS: Add "C:" for URI for chat where developers hang out Jani Nikula
@ 2016-10-20 12:22 ` Jani Nikula
2016-10-24 8:31 ` [PATCH 1/4] MAINTAINERS: Add "B:" for URI where to file bugs Daniel Vetter
` (2 subsequent siblings)
5 siblings, 0 replies; 10+ messages in thread
From: Jani Nikula @ 2016-10-20 12:22 UTC (permalink / raw)
To: linux-kernel
Cc: intel-gfx, dri-devel, daniel, airlied, joe, akpm, Jani Nikula
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
MAINTAINERS | 2 ++
1 file changed, 2 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 53df9ccc5573..a6bde481170c 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3954,6 +3954,7 @@ M: David Airlie <airlied@linux.ie>
L: dri-devel@lists.freedesktop.org
T: git git://people.freedesktop.org/~airlied/linux
B: https://bugs.freedesktop.org/
+C: irc://chat.freenode.net/dri-devel
S: Maintained
F: drivers/gpu/drm/
F: drivers/gpu/vga/
@@ -4007,6 +4008,7 @@ L: intel-gfx@lists.freedesktop.org
L: dri-devel@lists.freedesktop.org
W: https://01.org/linuxgraphics/
B: https://01.org/linuxgraphics/documentation/how-report-bugs
+C: irc://chat.freenode.net/intel-gfx
Q: http://patchwork.freedesktop.org/project/intel-gfx/
T: git git://anongit.freedesktop.org/drm-intel
S: Supported
--
2.1.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 1/4] MAINTAINERS: Add "B:" for URI where to file bugs
2016-10-20 12:22 [PATCH 1/4] MAINTAINERS: Add "B:" for URI where to file bugs Jani Nikula
` (2 preceding siblings ...)
2016-10-20 12:22 ` [PATCH 4/4] MAINTAINERS: add drm and drm/i915 irc channels Jani Nikula
@ 2016-10-24 8:31 ` Daniel Vetter
2016-10-24 8:48 ` Andrew Donnellan
2016-10-24 18:05 ` [RFC PATCH] get_maintainer: Look for arbitrary letter prefixes in sections Joe Perches
5 siblings, 0 replies; 10+ messages in thread
From: Daniel Vetter @ 2016-10-24 8:31 UTC (permalink / raw)
To: Jani Nikula
Cc: linux-kernel, intel-gfx, dri-devel, daniel, airlied, joe, akpm
On Thu, Oct 20, 2016 at 03:22:12PM +0300, Jani Nikula wrote:
> Different subsystems and drivers have different preferences for where to
> file bugs and what information to include. Add "B:" entry for specifying
> the URI for the bug tracker directly, a web page for detailed info on
> filing bugs, or a mailto: URI.
>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Joe Perches <joe@perches.com>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>
> ---
>
> Dust settled since the last time, maybe we can make URI work?
fwiw I want this, I think documenting bugzilla and irc channels (for
subsystems and drivers using them) will help a lot to make
(sub)communities of the kernel more approachable. On the entire series:
Very-much-Wanted-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> ---
> MAINTAINERS | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 89112a65b831..4a47ec00a09d 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -74,6 +74,8 @@ Descriptions of section entries:
> These reviewers should be CCed on patches.
> L: Mailing list that is relevant to this area
> W: Web-page with status/info
> + B: URI for where to file bugs. A web-page with detailed bug
> + filing info, a direct bug tracker link, or a mailto: URI.
> Q: Patchwork web based patch tracking system site
> T: SCM tree type and location.
> Type is one of: git, hg, quilt, stgit, topgit
> --
> 2.1.4
>
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/4] MAINTAINERS: Add "B:" for URI where to file bugs
2016-10-20 12:22 [PATCH 1/4] MAINTAINERS: Add "B:" for URI where to file bugs Jani Nikula
` (3 preceding siblings ...)
2016-10-24 8:31 ` [PATCH 1/4] MAINTAINERS: Add "B:" for URI where to file bugs Daniel Vetter
@ 2016-10-24 8:48 ` Andrew Donnellan
2016-10-24 18:05 ` [RFC PATCH] get_maintainer: Look for arbitrary letter prefixes in sections Joe Perches
5 siblings, 0 replies; 10+ messages in thread
From: Andrew Donnellan @ 2016-10-24 8:48 UTC (permalink / raw)
To: Jani Nikula, linux-kernel
Cc: intel-gfx, dri-devel, daniel, airlied, joe, akpm
On 20/10/16 23:22, Jani Nikula wrote:
> Different subsystems and drivers have different preferences for where to
> file bugs and what information to include. Add "B:" entry for specifying
> the URI for the bug tracker directly, a web page for detailed info on
> filing bugs, or a mailto: URI.
>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Joe Perches <joe@perches.com>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Both new fields seem like a very, very good idea to me. It'll definitely
save me a bit of googling and wiki-searching on the rare occasion I
venture outside powerpc...
For the whole series:
Reviewed-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com>
--
Andrew Donnellan OzLabs, ADL Canberra
andrew.donnellan@au1.ibm.com IBM Australia Limited
^ permalink raw reply [flat|nested] 10+ messages in thread
* [RFC PATCH] get_maintainer: Look for arbitrary letter prefixes in sections
2016-10-20 12:22 [PATCH 1/4] MAINTAINERS: Add "B:" for URI where to file bugs Jani Nikula
` (4 preceding siblings ...)
2016-10-24 8:48 ` Andrew Donnellan
@ 2016-10-24 18:05 ` Joe Perches
2016-11-03 9:07 ` Paul Bolle
5 siblings, 1 reply; 10+ messages in thread
From: Joe Perches @ 2016-10-24 18:05 UTC (permalink / raw)
To: Jani Nikula, linux-kernel; +Cc: intel-gfx, dri-devel, daniel, airlied, akpm
Jani Nikula proposes patches to add a few new letter prefixes
for "B:" bug reporting and "C:" maintainer chatting to the
various sections of MAINTAINERS.
Add a generic mechanism to get_maintainer.pl to find sections that
have any combination of "[A-Z]" letter prefix types in a section.
Signed-off-by: Joe Perches <joe@perches.com>
---
scripts/get_maintainer.pl | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index aed4511f0304..633f2dd3de27 100755
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -49,6 +49,7 @@ my $scm = 0;
my $web = 0;
my $subsystem = 0;
my $status = 0;
+my $letters = "";
my $keywords = 1;
my $sections = 0;
my $file_emails = 0;
@@ -241,6 +242,7 @@ if (!GetOptions(
'status!' => \$status,
'scm!' => \$scm,
'web!' => \$web,
+ 'letters=s' => \$letters,
'pattern-depth=i' => \$pattern_depth,
'k|keywords!' => \$keywords,
'sections!' => \$sections,
@@ -271,7 +273,8 @@ $output_multiline = 0 if ($output_separator ne ", ");
$output_rolestats = 1 if ($interactive);
$output_roles = 1 if ($output_rolestats);
-if ($sections) {
+if ($sections || $letters ne "") {
+ $sections = 1;
$email = 0;
$email_list = 0;
$scm = 0;
@@ -682,8 +685,10 @@ sub get_maintainers {
$line =~ s/\\\./\./g; ##Convert \. to .
$line =~ s/\.\*/\*/g; ##Convert .* to *
}
- $line =~ s/^([A-Z]):/$1:\t/g;
- print("$line\n");
+ my $count = $line =~ s/^([A-Z]):/$1:\t/g;
+ if ($letters eq "" || (!$count || $letters =~ /$1/i)) {
+ print("$line\n");
+ }
}
print("\n");
}
@@ -814,6 +819,7 @@ Other options:
--pattern-depth => Number of pattern directory traversals (default: 0 (all))
--keywords => scan patch for keywords (default: $keywords)
--sections => print all of the subsystem sections with pattern matches
+ --letters => print all matching 'letter' types from all matching sections
--mailmap => use .mailmap file (default: $email_use_mailmap)
--version => show version
--help => show this help information
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [RFC PATCH] get_maintainer: Look for arbitrary letter prefixes in sections
2016-10-24 18:05 ` [RFC PATCH] get_maintainer: Look for arbitrary letter prefixes in sections Joe Perches
@ 2016-11-03 9:07 ` Paul Bolle
2016-11-03 9:16 ` Joe Perches
0 siblings, 1 reply; 10+ messages in thread
From: Paul Bolle @ 2016-11-03 9:07 UTC (permalink / raw)
To: Joe Perches, Jani Nikula, linux-kernel
Cc: intel-gfx, dri-devel, daniel, airlied, akpm
On Mon, 2016-10-24 at 11:05 -0700, Joe Perches wrote:
> Jani Nikula proposes patches to add a few new letter prefixes
> for "B:" bug reporting and "C:" maintainer chatting to the
> various sections of MAINTAINERS.
>
> Add a generic mechanism to get_maintainer.pl to find sections that
> have any combination of "[A-Z]" letter prefix types in a section.
>
> Signed-off-by: Joe Perches <joe@perches.com>
This patch made it into linux-next (ie, next-20161028).
> --- a/scripts/get_maintainer.pl
> +++ b/scripts/get_maintainer.pl
> @@ -271,7 +273,8 @@ $output_multiline = 0 if ($output_separator ne ", ");
> $output_rolestats = 1 if ($interactive);
> $output_roles = 1 if ($output_rolestats);
>
> -if ($sections) {
> +if ($sections || $letters ne "") {
> + $sections = 1;
This triggers:
Unrecognized character \xA0; marked by <-- HERE after <-- HERE near column 1 at ./scripts/get_maintainer.pl line 277.
Git blame shows:
git blame -L 277,+1 ./scripts/get_maintainer.pl
b67071653d3fc (Joe Perches 2016-10-28 13:22:01 +1100 277) <A0><A0><A0><A0>$sections = 1;
(A0 seems to be the no break space. That character was inserted more
often further down the patch.)
Anybody else seeing this?
Paul Bolle
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [RFC PATCH] get_maintainer: Look for arbitrary letter prefixes in sections
2016-11-03 9:07 ` Paul Bolle
@ 2016-11-03 9:16 ` Joe Perches
2016-11-03 9:18 ` Paul Bolle
0 siblings, 1 reply; 10+ messages in thread
From: Joe Perches @ 2016-11-03 9:16 UTC (permalink / raw)
To: Paul Bolle, Jani Nikula, linux-kernel
Cc: intel-gfx, dri-devel, daniel, airlied, akpm
On Thu, 2016-11-03 at 10:07 +0100, Paul Bolle wrote:
> On Mon, 2016-10-24 at 11:05 -0700, Joe Perches wrote:
> > Jani Nikula proposes patches to add a few new letter prefixes
> > for "B:" bug reporting and "C:" maintainer chatting to the
> > various sections of MAINTAINERS.
> >
> > Add a generic mechanism to get_maintainer.pl to find sections that
> > have any combination of "[A-Z]" letter prefix types in a section.
> >
> > Signed-off-by: Joe Perches <joe@perches.com>
>
> This patch made it into linux-next (ie, next-20161028).
>
> > --- a/scripts/get_maintainer.pl
> > +++ b/scripts/get_maintainer.pl
> > @@ -271,7 +273,8 @@ $output_multiline = 0 if ($output_separator ne ", ");
> > $output_rolestats = 1 if ($interactive);
> > $output_roles = 1 if ($output_rolestats);
> >
> > -if ($sections) {
> > +if ($sections || $letters ne "") {
> > + $sections = 1;
>
> This triggers:
> Unrecognized character \xA0; marked by <-- HERE after <-- HERE near column 1 at ./scripts/get_maintainer.pl line 277.
>
> Git blame shows:
> git blame -L 277,+1 ./scripts/get_maintainer.pl
> b67071653d3fc (Joe Perches 2016-10-28 13:22:01 +1100 277) <A0><A0><A0><A0>$sections = 1;
>
> (A0 seems to be the no break space. That character was inserted more
> often further down the patch.)
>
> Anybody else seeing this?
Yes, it's been reported and should be fixed in -mm.
The fix should show up in -next in a little bit.
For now, try:
$ sed -i -e 's/\xA0/ /g' scripts/get_maintainer.pl
cheers, Joe
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [RFC PATCH] get_maintainer: Look for arbitrary letter prefixes in sections
2016-11-03 9:16 ` Joe Perches
@ 2016-11-03 9:18 ` Paul Bolle
0 siblings, 0 replies; 10+ messages in thread
From: Paul Bolle @ 2016-11-03 9:18 UTC (permalink / raw)
To: Joe Perches, Jani Nikula, linux-kernel
Cc: intel-gfx, dri-devel, daniel, airlied, akpm
On Thu, 2016-11-03 at 02:16 -0700, Joe Perches wrote:
> Yes, it's been reported and should be fixed in -mm.
> The fix should show up in -next in a little bit.
Great.
> For now, try:
> $ sed -i -e 's/\xA0/ /g' scripts/get_maintainer.pl
Thanks,
Paul Bolle
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2016-11-03 9:18 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-10-20 12:22 [PATCH 1/4] MAINTAINERS: Add "B:" for URI where to file bugs Jani Nikula
2016-10-20 12:22 ` [PATCH 2/4] MAINTAINERS: add drm and drm/i915 bug filing info Jani Nikula
2016-10-20 12:22 ` [PATCH 3/4] MAINTAINERS: Add "C:" for URI for chat where developers hang out Jani Nikula
2016-10-20 12:22 ` [PATCH 4/4] MAINTAINERS: add drm and drm/i915 irc channels Jani Nikula
2016-10-24 8:31 ` [PATCH 1/4] MAINTAINERS: Add "B:" for URI where to file bugs Daniel Vetter
2016-10-24 8:48 ` Andrew Donnellan
2016-10-24 18:05 ` [RFC PATCH] get_maintainer: Look for arbitrary letter prefixes in sections Joe Perches
2016-11-03 9:07 ` Paul Bolle
2016-11-03 9:16 ` Joe Perches
2016-11-03 9:18 ` Paul Bolle
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).