From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932319AbbJPSgQ (ORCPT ); Fri, 16 Oct 2015 14:36:16 -0400 Received: from mga03.intel.com ([134.134.136.65]:35290 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932093AbbJPSgP (ORCPT ); Fri, 16 Oct 2015 14:36:15 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.17,689,1437462000"; d="scan'208";a="828738145" From: Jani Nikula To: Joe Perches Cc: linux-kernel@vger.kernel.org Subject: Re: [PATCH] get_maintainer: add support for using an alternate MAINTAINERS file In-Reply-To: <1445012634.22921.34.camel@perches.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <1444984605-7445-1-git-send-email-jani.nikula@intel.com> <1444985457.22921.12.camel@perches.com> <878u73npjg.fsf@intel.com> <1445012634.22921.34.camel@perches.com> User-Agent: Notmuch/0.20.2+57~gff3a03d (http://notmuchmail.org) Emacs/24.4.1 (x86_64-pc-linux-gnu) Date: Fri, 16 Oct 2015 21:35:57 +0300 Message-ID: <87r3kumzjm.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 16 Oct 2015, Joe Perches wrote: > On Fri, 2015-10-16 at 12:14 +0300, Jani Nikula wrote: >> On Fri, 16 Oct 2015, Joe Perches wrote: >> > On Fri, 2015-10-16 at 11:36 +0300, Jani Nikula wrote: >> >> There are large and/or complex subsystems/drivers that have domain >> >> experts that should review patches in their domain. One such example is >> >> drm/i915. We'd like to be able to document this in a way that can be >> >> automatically queried for each patch, so people know who to ping for >> >> reviews. This is what get_maintainer.pl already solves. >> >> >> >> However, documenting all of this in the main kernel MAINTAINERS file is >> >> just too much noise, and potentially confusing for community >> >> contributors. Add support for specifying and using an alternate >> >> MAINTAINERS file with --maintainers option. >> > >> > Is this really useful for the community at large? >> >> Probably not. >> >> > This seems like something that might be useful for an >> > organization but not others. >> >> It may be useful for several organizations contributing to the kernel. >> >> > Why is specifying whatever is necessary in the existing >> > MAINTAINERS file noisy or confusing? >> >> IIUC you can't specify file patterns for specific reviewers within one >> entry. I think we'd have to split up the driver entry to several, mostly >> duplicated and possibly overlapping entries, with their own designated >> reviewers and file patterns. I think that would be noisy and confusing. > > I find the concept of adding separate MAINTAINERS files odd > and at best and not good for the community. Let me get this straight. You're rejecting a trivial patch increasing the usefulness of a simple script to a number of kernel developers not on technical grounds but because in your view the intended use is not good for the community? So had I said, this patch enables one to write unit tests for the script with various input files, the outcome might have been different? > Internal to an organization, if this is for subject matter > expert reviewers, perhaps it'd be better to add an optional > "REVIEWERS" file (or maybe multiple REVIEWER.* files) with > simpler patterns. It's fine if there's an option to specify additional reviewers files (that don't need to reside in the kernel tree) on the command line. Thanks for your time and review. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center