From: "Rafael J. Wysocki" <rjw@rjwysocki.net>
To: Jani Nikula <jani.nikula@intel.com>
Cc: linux-kernel@vger.kernel.org,
        "Rafael J. Wysocki" <rafael.j.wysocki@intel.com>,
        Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH] MAINTAINERS: clarify that "B:" is the URI where to file bugs
Date: Tue, 06 Dec 2016 23:10:28 +0100
Message-ID: <2136399.gq9DkmPDJp@aspire.rjw.lan>
User-Agent: KMail/4.14.10 (Linux/4.9.0-rc5+; KDE/4.14.9; x86_64; ; )
In-Reply-To: <1480939439-27947-1-git-send-email-jani.nikula@intel.com>
References: <1480939439-27947-1-git-send-email-jani.nikula@intel.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2343
Lines: 58

On Monday, December 05, 2016 02:03:59 PM Jani Nikula wrote:
> Different subsystems and drivers have different preferences for where to
> file bugs and what information to include. 686564434e88 ("MAINTAINERS:
> Add bug tracking system location entry type") added "B:" entry for this.
> 
> Clarify that "B:" specifies the URI for the bug tracker directly, a web
> page for detailed info on filing bugs, or a mailto: URI.
> 
> Fixes: 686564434e88 ("MAINTAINERS: Add bug tracking system location entry type")
> Cc: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> 
> ---
> 
> Rafael, I just noticed the "B:" entry popped up in MAINTAINERS from
> 686564434e88 ("MAINTAINERS: Add bug tracking system location entry
> type").
> 
> I've been pushing this for some time now, and I'd sent the last patch
> adding this before the kernel summit discussion you refer to [1], and
> Andrew picked it up, along with the rest in the series. This is where
> the whole idea came from!
> 
> Specifying "B:" as URI lets subsystems decide whether it contains a bug
> tracker or something else.
> 
> [1] http://lkml.kernel.org/r/1476966135-26943-1-git-send-email-jani.nikula@intel.com

I didn't realize that this was on the way in, sorry about that.

That said I'm slightly concerned about the last part of the modified description
below.  Namely, if mailing list information is already provided (the M: entry), it
obviously should be suitable for reporting bugs too, so I'm not sure about what
the "or a mailto: URI" role is?

> ---
>  MAINTAINERS | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index f7fd00ef1323..8f55d44b5b26 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -77,7 +77,8 @@ Descriptions of section entries:
>  	Q: Patchwork web based patch tracking system site
>  	T: SCM tree type and location.
>  	   Type is one of: git, hg, quilt, stgit, topgit
> -	B: Bug tracking system location.
> +	B: URI for where to file bugs. A web-page with detailed bug
> +	   filing info, a direct bug tracker link, or a mailto: URI.
>  	S: Status, one of the following:
>  	   Supported:	Someone is actually paid to look after this.
>  	   Maintained:	Someone actually looks after it.
> 

Thanks,
Rafael