Return-path: Received: from smtprelay0120.hostedemail.com ([216.40.44.120]:52433 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751284AbcHVSRi (ORCPT ); Mon, 22 Aug 2016 14:17:38 -0400 From: Joe Perches To: Andrew Morton , Kalle Valo , Arend van Spriel , Andy Whitcroft Cc: SF Markus Elfring , linux-wireless@vger.kernel.org, netdev@vger.kernel.org, Jouni Malinen , kernel-janitors@vger.kernel.org, Julia Lawall , linux-kernel@vger.kernel.org Subject: [PATCH] checkpatch: See if modified files are marked obsolete in MAINTAINERS Date: Mon, 22 Aug 2016 11:17:20 -0700 Message-Id: <7ceaa510dc9d2df05ec4b456baed7bb1415550b3.1471889575.git.joe@perches.com> (sfid-20160822_201802_346654_4503CD0F) In-Reply-To: <87shtwvm63.fsf@kamboji.qca.qualcomm.com> References: <87shtwvm63.fsf@kamboji.qca.qualcomm.com> Sender: linux-wireless-owner@vger.kernel.org List-ID: Use get_maintainer to check the status of individual files. If "obsolete", suggest leaving the files alone. Signed-off-by: Joe Perches --- scripts/checkpatch.pl | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 4de3cc4..df5e9d9 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -704,6 +704,16 @@ sub seed_camelcase_file { } } +sub is_maintained_obsolete { + my ($filename) = @_; + + return 0 if (!(-e "$root/scripts/get_maintainer.pl")); + + my $status = `perl $root/scripts/get_maintainer.pl --status --nom --nol --nogit --nogit-fallback $filename 2>&1`; + + return $status =~ /obsolete/i; +} + my $camelcase_seeded = 0; sub seed_camelcase_includes { return if ($camelcase_seeded); @@ -2289,6 +2299,10 @@ sub process { } if ($found_file) { + if (is_maintained_obsolete($realfile)) { + WARN("OBSOLETE", + "$realfile is marked as 'obsolete' in the MAINTAINERS hierarchy. No unnecessary modifications please.\n"); + } if ($realfile =~ m@^(?:drivers/net/|net/|drivers/staging/)@) { $check = 1; } else { -- 2.8.0.rc4.16.g56331f8