From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH] MAINTAINERS: Add "B:" preferred bug reporting method Date: Sat, 16 Jan 2016 13:39:53 +0200 Message-ID: <87lh7peos6.fsf@intel.com> References: <1452869104-22251-1-git-send-email-jani.nikula@intel.com> <1452903772.3699.4.camel@perches.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by gabe.freedesktop.org (Postfix) with ESMTP id 3FCD86E222 for ; Sat, 16 Jan 2016 03:40:00 -0800 (PST) In-Reply-To: <1452903772.3699.4.camel@perches.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Joe Perches , linux-kernel@vger.kernel.org Cc: intel-gfx@lists.freedesktop.org, Andrew Morton List-Id: intel-gfx@lists.freedesktop.org T24gU2F0LCAxNiBKYW4gMjAxNiwgSm9lIFBlcmNoZXMgPGpvZUBwZXJjaGVzLmNvbT4gd3JvdGU6 Cj4gT24gRnJpLCAyMDE2LTAxLTE1IGF0IDE2OjQ1ICswMjAwLCBKYW5pIE5pa3VsYSB3cm90ZToK Pj4gRGlmZmVyZW50IHN1YnN5c3RlbXMgYW5kIGRyaXZlcnMgaGF2ZSBkaWZmZXJlbnQgcHJlZmVy cmVkIHdheXMgb2YKPj4gcmVjZWl2aW5nIGJ1ZyByZXBvcnRzOyBtYWlsaW5nIGxpc3Qgb3IgYnVn emlsbGFzIGF0IHZhcmlvdXMKPj4gbG9jYXRpb25zLiBBZGQgIkI6IiBlbnRyeSBmb3Igc3BlY2lm eWluZyB0aGUgcHJlZmVyZW5jZSB0byBndWlkZSBidWcKPj4gcmVwb3J0ZXJzIGF0IHRoZSByaWdo dCBsb2NhdGlvbi4KPiBbXQo+PiBkaWZmIC0tZ2l0IGEvTUFJTlRBSU5FUlMgYi9NQUlOVEFJTkVS Uwo+IFtdCj4+IEBAIC03NSw2ICs3NSw4IEBAIERlc2NyaXB0aW9ucyBvZiBzZWN0aW9uIGVudHJp ZXM6Cj4+IMKgCUw6IE1haWxpbmcgbGlzdCB0aGF0IGlzIHJlbGV2YW50IHRvIHRoaXMgYXJlYQo+ PiDCoAlXOiBXZWItcGFnZSB3aXRoIHN0YXR1cy9pbmZvCj4+IMKgCVE6IFBhdGNod29yayB3ZWIg YmFzZWQgcGF0Y2ggdHJhY2tpbmcgc3lzdGVtIHNpdGUKPj4gKwlCOiBQcmVmZXJyZWQgbWV0aG9k IGZvciByZXBvcnRpbmcgYnVnczsgYnVnIHRyYWNraW5nIHN5c3RlbSBzaXRlCj4+ICsJwqDCoMKg b3IgIkxpc3QiIGZvciBtYWlsaW5nIGxpc3QuCj4+IMKgCVQ6IFNDTSB0cmVlIHR5cGUgYW5kIGxv Y2F0aW9uLgo+PiDCoAnCoMKgwqBUeXBlIGlzIG9uZSBvZjogZ2l0LCBoZywgcXVpbHQsIHN0Z2l0 LCB0b3BnaXQKPj4gwqAJUzogU3RhdHVzLCBvbmUgb2YgdGhlIGZvbGxvd2luZzoKPj4gQEAgLTM2 NTUsNiArMzY1Nyw3IEBAIEw6CWludGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKPj4gwqBM OglkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCj4+IMKgVzoJaHR0cHM6Ly8wMS5vcmcv bGludXhncmFwaGljcy8KPj4gwqBROglodHRwOi8vcGF0Y2h3b3JrLmZyZWVkZXNrdG9wLm9yZy9w cm9qZWN0L2ludGVsLWdmeC8KPj4gK0I6CWh0dHBzOi8vYnVncy5mcmVlZGVza3RvcC5vcmcvZW50 ZXJfYnVnLmNnaT9wcm9kdWN0PURSSSZjb21wb25lbnQ9RFJNL0ludGVsCj4KPiBUaGlzIHJlcXVp cmVzIGEgTG9naW5JRCAmIHBhc3N3b3JkCgpJdCBkb2VzLCBhbmQgSSBleHBlY3QgYmFzaWNhbGx5 IGFueSBidWcgcmVwb3J0aW5nIHRvb2wgdG8gZG8gc28uCgpJIHdhcyBkaXZpZGVkIGFib3V0IGxp bmtpbmcgdG8gdGhlIGZvcm0gcmVxdWlyaW5nIGxvZ2luIGRpcmVjdGx5LCBidXQKdGhpcyBoZWxw cyB0aGUgcmVwb3J0ZXIgdG8gdGhlIHJpZ2h0IHByb2R1Y3QgYW5kIGNvbXBvbmVudCwgd2hpY2gg bWF5Cm5vdCBiZSBvYnZpb3VzLiBBbnl0aGluZyB0byBtYWtlIHRoZSBidWcgcmVwb3J0ZXJzJyBs aXZlcyBlYXNpZXIuCgo+IE1heWJlIHVzZWZ1bCB0byBzaG93IHRoZSBvcGVuIGJ1Z3MgdG9vOgo+ IGh0dHBzOi8vYnVncy5mcmVlZGVza3RvcC5vcmcvYnVnbGlzdC5jZ2k/Y29tcG9uZW50PURSTSUy RkludGVsJnByb2R1Y3Q9RFJJJnJlc29sdXRpb249LS0tCgpJbiB0aGUgZW5kLCBJIGRvbid0IG1p bmQgd2hldGhlciBpdCdzIHRoZSBVUkwgSSBzdWdnZXN0ZWQsIHRoZSBVUkwgeW91CnN1Z2dlc3Rl ZCwgb3Igc29tZSBjb21iaW5hdGlvbiwgYXMgbG9uZyBhcyBpdCBkaXJlY3RzIHRoZSBidWcgcmVw b3J0ZXIKdG8gaHR0cHM6Ly9idWdzLmZyZWVkZXNrdG9wLm9yZy8uCgpUaGUgbWFpbiBpZGVhIGlz IGp1c3QgdG8gbWFrZSB0aGUgbGlmZSBvZiBidWcgcmVwb3J0ZXJzIGFuZCBtYWludGFpbmVycwpl YXNpZXIgYnkgZG9jdW1lbnRpbmcgd2hlcmUgdGhlIG1haW50YWluZXJzIHdhbnQgdGhlIGJ1Z3Mg cmVwb3J0ZWQuCgpCUiwKSmFuaS4KCj4KPiBNYXliZSB0aGUgZ2V0X21haW50YWluZXIgc2NyaXB0 IHNob3VsZCBiZSB1cGRhdGVkLgo+IFNvbWV0aGluZyBsaWtlOiAodW50ZXN0ZWQpCj4gLS0tCj4g wqBzY3JpcHRzL2dldF9tYWludGFpbmVyLnBsIHwgMTcgKysrKysrKysrKysrKysrLS0KPiDCoDEg ZmlsZSBjaGFuZ2VkLCAxNSBpbnNlcnRpb25zKCspLCAyIGRlbGV0aW9ucygtKQo+Cj4gZGlmZiAt LWdpdCBhL3NjcmlwdHMvZ2V0X21haW50YWluZXIucGwgYi9zY3JpcHRzL2dldF9tYWludGFpbmVy LnBsCj4gaW5kZXggMTg3MzQyMS4uYmJlNTMzNyAxMDA3NTUKPiAtLS0gYS9zY3JpcHRzL2dldF9t YWludGFpbmVyLnBsCj4gKysrIGIvc2NyaXB0cy9nZXRfbWFpbnRhaW5lci5wbAo+IEBAIC00Nyw2 ICs0Nyw3IEBAIG15ICRvdXRwdXRfcm9sZXN0YXRzID0gMTsKPiDCoG15ICRvdXRwdXRfc2VjdGlv bl9tYXhsZW4gPSA1MDsKPiDCoG15ICRzY20gPSAwOwo+IMKgbXkgJHdlYiA9IDA7Cj4gK215ICRi dWcgPSAwOwo+IMKgbXkgJHN1YnN5c3RlbSA9IDA7Cj4gwqBteSAkc3RhdHVzID0gMDsKPiDCoG15 ICRrZXl3b3JkcyA9IDE7Cj4gQEAgLTIzOSw2ICsyNDAsNyBAQCBpZiAoIUdldE9wdGlvbnMoCj4g wqAJCSdzdGF0dXMhJyA9PiBcJHN0YXR1cywKPiDCoAkJJ3NjbSEnID0+IFwkc2NtLAo+IMKgCQkn d2ViIScgPT4gXCR3ZWIsCj4gKwkJJ2J1ZyEnID0+IFwkYnVnLAo+IMKgCQkncGF0dGVybi1kZXB0 aD1pJyA9PiBcJHBhdHRlcm5fZGVwdGgsCj4gwqAJCSdrfGtleXdvcmRzIScgPT4gXCRrZXl3b3Jk cywKPiDCoAkJJ3NlY3Rpb25zIScgPT4gXCRzZWN0aW9ucywKPiBAQCAtMjc2LDEyICsyNzgsMTMg QEAgaWYgKCRzZWN0aW9ucykgewo+IMKgwqDCoMKgwqAkc3RhdHVzID0gMDsKPiDCoMKgwqDCoMKg JHN1YnN5c3RlbSA9IDA7Cj4gwqDCoMKgwqDCoCR3ZWIgPSAwOwo+ICvCoMKgwqDCoCRidWcgPSAw Owo+IMKgwqDCoMKgwqAka2V5d29yZHMgPSAwOwo+IMKgwqDCoMKgwqAkaW50ZXJhY3RpdmUgPSAw Owo+IMKgfSBlbHNlIHsKPiAtwqDCoMKgwqBteSAkc2VsZWN0aW9ucyA9ICRlbWFpbCArICRzY20g KyAkc3RhdHVzICsgJHN1YnN5c3RlbSArICR3ZWI7Cj4gK8KgwqDCoMKgbXkgJHNlbGVjdGlvbnMg PSAkZW1haWwgKyAkc2NtICsgJHN0YXR1cyArICRzdWJzeXN0ZW0gKyAkd2ViICsgJGJ1ZzsKPiDC oMKgwqDCoMKgaWYgKCRzZWxlY3Rpb25zID09IDApIHsKPiAtCWRpZSAiJFA6wqDCoE1pc3Npbmcg cmVxdWlyZWQgb3B0aW9uOiBlbWFpbCwgc2NtLCBzdGF0dXMsIHN1YnN5c3RlbSBvciB3ZWJcbiI7 Cj4gKwlkaWUgIiRQOsKgwqBNaXNzaW5nIHJlcXVpcmVkIG9wdGlvbjogZW1haWwsIHNjbSwgc3Rh dHVzLCBzdWJzeXN0ZW0sIHdlYiBvciBidWdcbiI7Cj4gwqDCoMKgwqDCoH0KPiDCoH0KPiDCoAo+ IEBAIC01MDUsNiArNTA4LDcgQEAgbXkgJWhhc2hfbGlzdF90bzsKPiDCoG15IEBsaXN0X3RvID0g KCk7Cj4gwqBteSBAc2NtID0gKCk7Cj4gwqBteSBAd2ViID0gKCk7Cj4gK215IEBidWcgPSAoKTsK PiDCoG15IEBzdWJzeXN0ZW0gPSAoKTsKPiDCoG15IEBzdGF0dXMgPSAoKTsKPiDCoG15ICVkZWR1 cGxpY2F0ZV9uYW1lX2hhc2ggPSAoKTsKPiBAQCAtNTM3LDYgKzU0MSwxMSBAQCBpZiAoJHdlYikg ewo+IMKgwqDCoMKgwqBvdXRwdXQoQHdlYik7Cj4gwqB9Cj4gwqAKPiAraWYgKCRidWcpIHsKPiAr wqDCoMKgwqBAYnVnID0gdW5pcShAYnVnKTsKPiArwqDCoMKgwqBvdXRwdXQoQGJ1Zyk7Cj4gK30K PiArCj4gwqBleGl0KCRleGl0KTsKPiDCoAo+IMKgc3ViIGlnbm9yZV9lbWFpbF9hZGRyZXNzIHsK PiBAQCAtNTkzLDYgKzYwMiw3IEBAIHN1YiBnZXRfbWFpbnRhaW5lcnMgewo+IMKgwqDCoMKgwqBA bGlzdF90byA9ICgpOwo+IMKgwqDCoMKgwqBAc2NtID0gKCk7Cj4gwqDCoMKgwqDCoEB3ZWIgPSAo KTsKPiArwqDCoMKgwqBAYnVnID0gKCk7Cj4gwqDCoMKgwqDCoEBzdWJzeXN0ZW0gPSAoKTsKPiDC oMKgwqDCoMKgQHN0YXR1cyA9ICgpOwo+IMKgwqDCoMKgwqAlZGVkdXBsaWNhdGVfbmFtZV9oYXNo ID0gKCk7Cj4gQEAgLTgwMiw2ICs4MTIsNyBAQCBNQUlOVEFJTkVSIGZpZWxkIHNlbGVjdGlvbiBv cHRpb25zOgo+IMKgwqDCoC0tc3RhdHVzID0+IHByaW50IHN0YXR1cyBpZiBhbnkKPiDCoMKgwqAt LXN1YnN5c3RlbSA9PiBwcmludCBzdWJzeXN0ZW0gbmFtZSBpZiBhbnkKPiDCoMKgwqAtLXdlYiA9 PiBwcmludCB3ZWJzaXRlKHMpIGlmIGFueQo+ICvCoMKgLS1idWcgPT4gcHJpbnQgYnVnIHJlcG9y dGluZyBtZWNoYW5pc20ocykgaWYgYW55Cj4gwqAKPiDCoE91dHB1dCB0eXBlIG9wdGlvbnM6Cj4g wqDCoMKgLS1zZXBhcmF0b3IgWywgXSA9PiBzZXBhcmF0b3IgZm9yIG11bHRpcGxlIGVudHJpZXMg b24gMSBsaW5lCj4gQEAgLTExMjksNiArMTE0MCw4IEBAIHN1YiBhZGRfY2F0ZWdvcmllcyB7Cj4g wqAJCXB1c2goQHNjbSwgJHB2YWx1ZSk7Cj4gwqAJwqDCoMKgwqB9IGVsc2lmICgkcHR5cGUgZXEg IlciKSB7Cj4gwqAJCXB1c2goQHdlYiwgJHB2YWx1ZSk7Cj4gKwnCoMKgwqDCoH0gZWxzaWYgKCRw dHlwZSBlcSAiQiIpIHsKPiArCQlwdXNoKEBidWcsICRwdmFsdWUpOwo+IMKgCcKgwqDCoMKgfSBl bHNpZiAoJHB0eXBlIGVxICJTIikgewo+IMKgCQlwdXNoKEBzdGF0dXMsICRwdmFsdWUpOwo+IMKg CcKgwqDCoMKgfQoKLS0gCkphbmkgTmlrdWxhLCBJbnRlbCBPcGVuIFNvdXJjZSBUZWNobm9sb2d5 IENlbnRlcgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpJ bnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0 cDovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2ludGVsLWdmeAo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751651AbcAPLkD (ORCPT ); Sat, 16 Jan 2016 06:40:03 -0500 Received: from mga01.intel.com ([192.55.52.88]:17327 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750877AbcAPLkA convert rfc822-to-8bit (ORCPT ); Sat, 16 Jan 2016 06:40:00 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,304,1449561600"; d="scan'208";a="894433545" From: Jani Nikula To: Joe Perches , linux-kernel@vger.kernel.org Cc: intel-gfx@lists.freedesktop.org, Daniel Vetter , Andrew Morton Subject: Re: [PATCH] MAINTAINERS: Add "B:" preferred bug reporting method In-Reply-To: <1452903772.3699.4.camel@perches.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <1452869104-22251-1-git-send-email-jani.nikula@intel.com> <1452903772.3699.4.camel@perches.com> User-Agent: Notmuch/0.21+33~g570c0ae (http://notmuchmail.org) Emacs/24.4.1 (x86_64-pc-linux-gnu) Date: Sat, 16 Jan 2016 13:39:53 +0200 Message-ID: <87lh7peos6.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, 16 Jan 2016, Joe Perches 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@lists.freedesktop.org >>  L: dri-devel@lists.freedesktop.org >>  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