From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751552AbdGWUFf (ORCPT ); Sun, 23 Jul 2017 16:05:35 -0400 Received: from smtprelay0121.hostedemail.com ([216.40.44.121]:39747 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751423AbdGWUFd (ORCPT ); Sun, 23 Jul 2017 16:05:33 -0400 X-Session-Marker: 6A6F6540706572636865732E636F6D X-Spam-Summary: 50,0,0,,d41d8cd98f00b204,joe@perches.com,:::::::,RULES_HIT:41:355:379:541:560:599:800:960:967:973:982:988:989:1260:1277:1311:1313:1314:1345:1359:1373:1437:1515:1516:1518:1534:1542:1593:1594:1711:1730:1747:1777:1792:2198:2199:2376:2393:2525:2540:2560:2563:2682:2685:2828:2859:2933:2937:2939:2942:2945:2947:2951:2954:3022:3138:3139:3140:3141:3142:3354:3622:3865:3866:3867:3868:3870:3871:3872:3873:3874:3934:3936:3938:3941:3944:3947:3950:3953:3956:3959:4250:4321:5007:6119:7809:7903:8985:9025:10004:10400:10848:11232:11658:11783:11914:12043:12296:12555:12663:12740:12895:13439:13894:14181:14659:14721:21080:21451:21627:30054:30083:30091,0,RBL:none,CacheIP:none,Bayesian:0.5,0.5,0.5,Netcheck:none,DomainCache:0,MSF:not bulk,SPF:,MSBL:0,DNSBL:none,Custom_rules:0:0:0,LFtime:2,LUA_SUMMARY:none X-HE-Tag: route73_30518ddd0e32e X-Filterd-Recvd-Size: 3138 Message-ID: <1500840330.23924.1.camel@perches.com> Subject: Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering From: Joe Perches To: Linus Torvalds , Randy Dunlap Cc: LKML , Andrew Morton Date: Sun, 23 Jul 2017 13:05:30 -0700 In-Reply-To: References: Content-Type: text/plain; charset="ISO-8859-1" X-Mailer: Evolution 3.22.6-1ubuntu1 Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, 2017-07-23 at 12:49 -0700, Linus Torvalds wrote: > Ok, so I already applied your alpha-ordering patch, but it just annoyed me that > > (a) the ordering wasn't complete > > (b) this wasn't scripted. > > However, the sane way of scripting it is clearly not to do it in C, > which I'd be comfy with, because that would be insane. > > Instead, it should be done in perl. Except my perl-fu is so horribly > horribly bad that I'm a bit ashamed to show the end result. > > Does anybody have actual real perl skills? Because somebody should > double-check my appended script-from-hell. > > ANYWAY. One reason I did this was because *if* we want to split up the > MAINTAINERS file, I absolutely refuse to do it by hand. It needs to be > automated. I'm not going to apply a patch - I'm going to apply a > *script*, and commit the end result along with the doc about what the > script was (so that then I have an inevitable conflict due to this big > re-org, I can resolve the conflict by re-running the script on the > side that wasn't part of the re-org, rather than having to do nasty > things). > > And this script could easily be extended to automate the scripting. So > please, can somebody with perl-fu say that "yeah, that's the right > perl model", or point me to what I did wrong? > > The end result looks ok. I can run > > perl parse-maintainers.pl < MAINTAINERS > outfile > > and the end result is actually a *properly* sorted MAINTAINERS file as > far as I can tell. > > Comments? That works OK except for this section where there are 2 header lines EDAC-XGENE APPLIED MICRO (APM) X-GENE SOC EDAC M:     Loc Ho S:     Supported F:     drivers/edac/xgene_edac.c F:     Documentation/devicetree/bindings/edac/apm-xgene-edac.txt If you take up the patch I sent for that before you run the script, it should be OK. https://patchwork.kernel.org/patch/9857337/ I'll send a get_maintainers patch that allows a few different styles of MAINTAINERS files separately. o A single top level MAINTAINERS file o A MAINTAINERS directory with multiple section files o MAINTAINERS files distributed around the kernel source tree