Re: [Ksummit-discuss] [RFC PATCH 2/3] MAINTAINERS, Handbook: Subsystem Profile

2018-11-19 Thread Jani Nikula
On Fri, 16 Nov 2018, Joe Perches  wrote:
> On Fri, 2018-11-16 at 14:44 +0200, Jani Nikula wrote:
>> I quickly cooked up this script to produce the top-5 commit prefixes for
>> the given files over the arbitrary last 200 commits. It'll give you a
>> pretty good idea if you're even close.
>> 
>> ---
>> #!/bin/sh
>> # usage: subject-prefix FILE [...]
>> # show top 5 subject prefixes for FILEs
>> 
>> git log --format=%s -n 200 -- "$@" |\
>>  grep -v "^Merge " |\
>>  sed 's/\(.*\):.*/\1/' |\
>>  sort | uniq -c | sort -nr | sed 's/ *[0-9]\+ //' |\
>>  head -n 5
>> ---
>> 
>> Someone who knows perl could turn that into a checkpatch check: See if
>> the patch subject prefix is one of the top-5 for all files changed by
>> the patch, and ask the user to double check if it isn't. Or some
>> heuristics thereof.
>
> This won't work when a patch contains multiple files
> from different paths, or even multiple files from a
> single driver.

*shrug*

You can give it multiple files as argument, and it'll give you an
approximation of what the prefix could be, whether you're way off or
not. Close enough at least for the single driver case. Obviously not
perfect, but hey, it took me all of five minutes to write that. ;)

BR,
Jani.


-- 
Jani Nikula, Intel Open Source Graphics Center


Re: [Ksummit-discuss] [RFC PATCH 2/3] MAINTAINERS, Handbook: Subsystem Profile

2018-11-18 Thread Mauro Carvalho Chehab
Em Sun, 18 Nov 2018 09:34:01 -0800
Dan Williams  escreveu:

> On Sun, Nov 18, 2018 at 9:31 AM Dan Williams  wrote:
> > On Sun, Nov 18, 2018 at 9:31 AM Dan Williams  
> > wrote:  
> [..]
> > > True, will remove. What's the point of stating daily active hours when we 
> > > already have "Resubmit Cadence" (I think I'll rename it "Follow Cadence") 
> > > measured in multiple days / weeks.  
> 
> Apologies, fumble-fingered that last mail. I meant to say "Follow-up
> Cadence", and that include a maintainer preference for private pings,
> or full patch set to be sent again to the list.

Yeah, "Follow-up Cadence" sounds a way better.

Thanks,
Mauro