linux-kernel - [PATCH] get_maintainer: add support for using an alternate MAINTAINERS file

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-Id: <1444984605-7445-1-git-send-email-jani.nikula@intel.com>
Date:	Fri, 16 Oct 2015 11:36:45 +0300
From:	Jani Nikula <jani.nikula@...el.com>
To:	linux-kernel@...r.kernel.org, Joe Perches <joe@...ches.com>
Cc:	jani.nikula@...el.com
Subject: [PATCH] get_maintainer: add support for using an alternate MAINTAINERS file

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.

Signed-off-by: Jani Nikula <jani.nikula@...el.com>

---

Note that I don't know perl, at all. This is all cargo culted copy
paste, but seems to work. Please be gentle. :)
---
 scripts/get_maintainer.pl | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index 98bae869f6d0..0dae244f26eb 100755
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -18,6 +18,7 @@ my $V = '0.26';
 use Getopt::Long qw(:config no_auto_abbrev);
 
 my $lk_path = "./";
+my $maintainers_path = "${lk_path}MAINTAINERS";
 my $email = 1;
 my $email_usename = 1;
 my $email_maintainer = 1;
@@ -229,6 +230,7 @@ if (!GetOptions(
 		'n!' => \$email_usename,
 		'l!' => \$email_list,
 		's!' => \$email_subscriber_list,
+		'maintainers=s' => \$maintainers_path,
 		'multiline!' => \$output_multiline,
 		'roles!' => \$output_roles,
 		'rolestats!' => \$output_rolestats,
@@ -300,8 +302,8 @@ if (!top_of_kernel_tree($lk_path)) {
 my @typevalue = ();
 my %keyword_hash;
 
-open (my $maint, '<', "${lk_path}MAINTAINERS")
-    or die "$P: Can't open MAINTAINERS: $!\n";
+open (my $maint, '<', $maintainers_path)
+    or die "$P: Can't open ${maintainers_path}: $!\n";
 while (<$maint>) {
     my $line = $_;
 
@@ -808,6 +810,7 @@ Other options:
   --keywords => scan patch for keywords (default: $keywords)
   --sections => print all of the subsystem sections with pattern matches
   --mailmap => use .mailmap file (default: $email_use_mailmap)
+  --maintainers => specify alternate MAINTAINERS file to use
   --version => show version
   --help => show this help information
 
-- 
2.1.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/