Re: [PATCH] MAINTAINERS: Add "B:" preferred bug reporting method

[Jani Nikula]

On Sat, 16 Jan 2016, Joe Perches <joe@xxxxxxxxxxx> wrote:
> On Fri, 2016-01-15 at 16:45 +0200, Jani Nikula wrote:
>> Different subsystems and drivers have different preferred ways of
>> receiving bug reports; mailing list or bugzillas at various
>> locations. Add "B:" entry for specifying the preference to guide bug
>> reporters at the right location.
> []
>> diff --git a/MAINTAINERS b/MAINTAINERS
> []
>> @@ -75,6 +75,8 @@ Descriptions of section entries:
>> Â	L: Mailing list that is relevant to this area
>> Â	W: Web-page with status/info
>> Â	Q: Patchwork web based patch tracking system site
>> +	B: Preferred method for reporting bugs; bug tracking system site
>> +	ÂÂÂor "List" for mailing list.
>> Â	T: SCM tree type and location.
>> Â	ÂÂÂType is one of: git, hg, quilt, stgit, topgit
>> Â	S: Status, one of the following:
>> @@ -3655,6 +3657,7 @@ L:	intel-gfx@xxxxxxxxxxxxxxxxxxxxx
>> ÂL:	dri-devel@xxxxxxxxxxxxxxxxxxxxx
>> ÂW:	https://01.org/linuxgraphics/
>> ÂQ:	http://patchwork.freedesktop.org/project/intel-gfx/
>> +B:	https://bugs.freedesktop.org/enter_bug.cgi?product=DRI&component=DRM/Intel
>
> This requires a LoginID & password

It does, and I expect basically any bug reporting tool to do so.

I was divided about linking to the form requiring login directly, but
this helps the reporter to the right product and component, which may
not be obvious. Anything to make the bug reporters' lives easier.

> Maybe useful to show the open bugs too:
> https://bugs.freedesktop.org/buglist.cgi?component=DRM%2FIntel&product=DRI&resolution=---

In the end, I don't mind whether it's the URL I suggested, the URL you
suggested, or some combination, as long as it directs the bug reporter
to https://bugs.freedesktop.org/.

The main idea is just to make the life of bug reporters and maintainers
easier by documenting where the maintainers want the bugs reported.

BR,
Jani.

>
> Maybe the get_maintainer script should be updated.
> Something like: (untested)
> ---
> Âscripts/get_maintainer.pl | 17 +++++++++++++++--
> Â1 file changed, 15 insertions(+), 2 deletions(-)
>
> diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
> index 1873421..bbe5337 100755
> --- a/scripts/get_maintainer.pl
> +++ b/scripts/get_maintainer.pl
> @@ -47,6 +47,7 @@ my $output_rolestats = 1;
> Âmy $output_section_maxlen = 50;
> Âmy $scm = 0;
> Âmy $web = 0;
> +my $bug = 0;
> Âmy $subsystem = 0;
> Âmy $status = 0;
> Âmy $keywords = 1;
> @@ -239,6 +240,7 @@ if (!GetOptions(
> Â		'status!' => \$status,
> Â		'scm!' => \$scm,
> Â		'web!' => \$web,
> +		'bug!' => \$bug,
> Â		'pattern-depth=i' => \$pattern_depth,
> Â		'k|keywords!' => \$keywords,
> Â		'sections!' => \$sections,
> @@ -276,12 +278,13 @@ if ($sections) {
> ÂÂÂÂÂ$status = 0;
> ÂÂÂÂÂ$subsystem = 0;
> ÂÂÂÂÂ$web = 0;
> +ÂÂÂÂ$bug = 0;
> ÂÂÂÂÂ$keywords = 0;
> ÂÂÂÂÂ$interactive = 0;
> Â} else {
> -ÂÂÂÂmy $selections = $email + $scm + $status + $subsystem + $web;
> +ÂÂÂÂmy $selections = $email + $scm + $status + $subsystem + $web + $bug;
> ÂÂÂÂÂif ($selections == 0) {
> -	die "$P:ÂÂMissing required option: email, scm, status, subsystem or web\n";
> +	die "$P:ÂÂMissing required option: email, scm, status, subsystem, web or bug\n";
> ÂÂÂÂÂ}
> Â}
> Â
> @@ -505,6 +508,7 @@ my %hash_list_to;
> Âmy @list_to = ();
> Âmy @scm = ();
> Âmy @web = ();
> +my @bug = ();
> Âmy @subsystem = ();
> Âmy @status = ();
> Âmy %deduplicate_name_hash = ();
> @@ -537,6 +541,11 @@ if ($web) {
> ÂÂÂÂÂoutput(@web);
> Â}
> Â
> +if ($bug) {
> +ÂÂÂÂ@bug = uniq(@bug);
> +ÂÂÂÂoutput(@bug);
> +}
> +
> Âexit($exit);
> Â
> Âsub ignore_email_address {
> @@ -593,6 +602,7 @@ sub get_maintainers {
> ÂÂÂÂÂ@list_to = ();
> ÂÂÂÂÂ@scm = ();
> ÂÂÂÂÂ@web = ();
> +ÂÂÂÂ@bug = ();
> ÂÂÂÂÂ@subsystem = ();
> ÂÂÂÂÂ@status = ();
> ÂÂÂÂÂ%deduplicate_name_hash = ();
> @@ -802,6 +812,7 @@ MAINTAINER field selection options:
> ÂÂÂ--status => print status if any
> ÂÂÂ--subsystem => print subsystem name if any
> ÂÂÂ--web => print website(s) if any
> +ÂÂ--bug => print bug reporting mechanism(s) if any
> Â
> ÂOutput type options:
> ÂÂÂ--separator [, ] => separator for multiple entries on 1 line
> @@ -1129,6 +1140,8 @@ sub add_categories {
> Â		push(@scm, $pvalue);
> Â	ÂÂÂÂ} elsif ($ptype eq "W") {
> Â		push(@web, $pvalue);
> +	ÂÂÂÂ} elsif ($ptype eq "B") {
> +		push(@bug, $pvalue);
> Â	ÂÂÂÂ} elsif ($ptype eq "S") {
> Â		push(@status, $pvalue);
> Â	ÂÂÂÂ}

-- 
Jani Nikula, Intel Open Source Technology Center