Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Fri, 2017-07-28 at 15:30 -0700, Linus Torvalds wrote: > On Thu, Jul 27, 2017 at 8:12 PM, Joe Perches wrote: > > > > I think it's better to centralize the MAINTAINERS > > location in /MAINTAINERS/ than spread > > them all over the tree given how many subsystems and > >

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Fri, 2017-07-28 at 15:30 -0700, Linus Torvalds wrote: > On Thu, Jul 27, 2017 at 8:12 PM, Joe Perches wrote: > > > > I think it's better to centralize the MAINTAINERS > > location in /MAINTAINERS/ than spread > > them all over the tree given how many subsystems and > > maintainerships are also

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Fri, 2017-07-28 at 15:30 -0700, Linus Torvalds wrote: > So we'd have files like "rdma", "dma", "omap", "pm", "dri", "pci", > "wireless" etc, all of which sound sane. [] > it looks like a promising approach to me, and I like how > the names seem to end up all fairly sane. > > Comments? Seems

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Fri, 2017-07-28 at 15:30 -0700, Linus Torvalds wrote: > So we'd have files like "rdma", "dma", "omap", "pm", "dri", "pci", > "wireless" etc, all of which sound sane. [] > it looks like a promising approach to me, and I like how > the names seem to end up all fairly sane. > > Comments? Seems

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Thu, Jul 27, 2017 at 8:12 PM, Joe Perches wrote: > > I think it's better to centralize the MAINTAINERS > location in /MAINTAINERS/ than spread > them all over the tree given how many subsystems and > maintainerships are also spread around the tree. > > But the get_maintainers

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Thu, Jul 27, 2017 at 8:12 PM, Joe Perches wrote: > > I think it's better to centralize the MAINTAINERS > location in /MAINTAINERS/ than spread > them all over the tree given how many subsystems and > maintainerships are also spread around the tree. > > But the get_maintainers patch I sent

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Thu, 2017-07-27 at 19:43 -0700, Linus Torvalds wrote: > On Thu, Jul 27, 2017 at 5:30 PM, Joe Perches wrote: > > > > Maybe add a reordering of the patterns so that each pattern list > > is in a specific order too > > I don't think this is wrong per se, but I'm not sure I

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Thu, 2017-07-27 at 19:43 -0700, Linus Torvalds wrote: > On Thu, Jul 27, 2017 at 5:30 PM, Joe Perches wrote: > > > > Maybe add a reordering of the patterns so that each pattern list > > is in a specific order too > > I don't think this is wrong per se, but I'm not sure I want to get > into

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Thu, Jul 27, 2017 at 5:30 PM, Joe Perches wrote: > > Maybe add a reordering of the patterns so that each pattern list > is in a specific order too I don't think this is wrong per se, but I'm not sure I want to get into the merge hell any more than we are already. Maybe

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Thu, Jul 27, 2017 at 5:30 PM, Joe Perches wrote: > > Maybe add a reordering of the patterns so that each pattern list > is in a specific order too I don't think this is wrong per se, but I'm not sure I want to get into the merge hell any more than we are already. Maybe when/if that file is

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Sun, 2017-07-23 at 16:14 -0700, Linus Torvalds wrote: > On Sun, Jul 23, 2017 at 1:00 PM, Linus Torvalds > wrote: > > > > Anyway, clearly my script showed something. I think my script is still > > doing the right thing, it's just that the input is questionable. >

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Sun, 2017-07-23 at 16:14 -0700, Linus Torvalds wrote: > On Sun, Jul 23, 2017 at 1:00 PM, Linus Torvalds > wrote: > > > > Anyway, clearly my script showed something. I think my script is still > > doing the right thing, it's just that the input is questionable. > > I added a few actual checks

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Sun, 2017-07-23 at 16:14 -0700, Linus Torvalds wrote: > On Sun, Jul 23, 2017 at 1:00 PM, Linus Torvalds > wrote: > > > > Anyway, clearly my script showed something. I think my script is still > > doing the right thing, it's just that the input is questionable. >

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Sun, 2017-07-23 at 16:14 -0700, Linus Torvalds wrote: > On Sun, Jul 23, 2017 at 1:00 PM, Linus Torvalds > wrote: > > > > Anyway, clearly my script showed something. I think my script is still > > doing the right thing, it's just that the input is questionable. > > I added a few actual checks

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Sun, Jul 23, 2017 at 1:00 PM, Linus Torvalds wrote: > > Anyway, clearly my script showed something. I think my script is still > doing the right thing, it's just that the input is questionable. I added a few actual checks for the error cases to the script, fixed

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Sun, Jul 23, 2017 at 1:00 PM, Linus Torvalds wrote: > > Anyway, clearly my script showed something. I think my script is still > doing the right thing, it's just that the input is questionable. I added a few actual checks for the error cases to the script, fixed up the problems, and committed

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Sun, 2017-07-23 at 13:00 -0700, Linus Torvalds wrote: > it looks like there were four copies of "GREYBUS PROTOCOLS > DRIVERS". WTF? I didn't check if there was something else odd going > on. > > I guess they'd need to be made unique somehow too. right. Unfortunate about the duplicate section

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Sun, 2017-07-23 at 13:00 -0700, Linus Torvalds wrote: > it looks like there were four copies of "GREYBUS PROTOCOLS > DRIVERS". WTF? I didn't check if there was something else odd going > on. > > I guess they'd need to be made unique somehow too. right. Unfortunate about the duplicate section

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

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

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

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

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Sun, Jul 23, 2017 at 12:49 PM, Linus Torvalds wrote: > > 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. Yeah, there's

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Sun, Jul 23, 2017 at 12:49 PM, Linus Torvalds wrote: > > 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. Yeah, there's something wrong there. I end up

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

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

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

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

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Fri, Jul 21, 2017 at 1:32 PM, Randy Dunlap wrote: > > and send with correct file encoding! Congratulations, you were indeed successful in fixing whatever locale issue that was biting you. Linus

Re: [PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

On Fri, Jul 21, 2017 at 1:32 PM, Randy Dunlap wrote: > > and send with correct file encoding! Congratulations, you were indeed successful in fixing whatever locale issue that was biting you. Linus

[PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

From: Randy Dunlap Fix major alphabetic errors. No attempt to fix items that all begin with the same word (like ARM, BROADCOM, DRM, EDAC, FREESCALE, INTEL, OMAP, PCI, SAMSUNG, TI, USB, etc.). (diffstat +/- is different by one line because TI KEYSTONE MULTICORE had 2 blank

[PATCH Y.A. RESEND] MAINTAINERS: fix alpha. ordering

From: Randy Dunlap Fix major alphabetic errors. No attempt to fix items that all begin with the same word (like ARM, BROADCOM, DRM, EDAC, FREESCALE, INTEL, OMAP, PCI, SAMSUNG, TI, USB, etc.). (diffstat +/- is different by one line because TI KEYSTONE MULTICORE had 2 blank lines after it.)