[U-Boot] [PATCH] patman: Don't request full names from get_maintainer

[Doug Anderson]

Simon,

On Fri, Apr 18, 2014 at 1:43 PM, Simon Glass <sjg at chromium.org> wrote:
> Hi Doug,
>
> On 17 April 2014 12:47, Doug Anderson <dianders at chromium.org> wrote:
>>
>> The Linux get_maintainer.pl can often produce a whole lot of results.
>> As a result you'll sometimes blow your CC field over 1024 characters
>> and that can cause listservs to reject your message.
>>
>> As a stopgap, call get_maintainer.pl with "--non" so it doesn't
>> include real names.  This will dramatically reduce the number of
>> characters.
>>
>> Signed-off-by: Doug Anderson <dianders at chromium.org>
>> ---
>>
>>  tools/patman/get_maintainer.py | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/tools/patman/get_maintainer.py b/tools/patman/get_maintainer.py
>> index 00b4939..a5160bc 100644
>> --- a/tools/patman/get_maintainer.py
>> +++ b/tools/patman/get_maintainer.py
>> @@ -43,5 +43,5 @@ def GetMaintainer(fname, verbose=False):
>>              print "WARNING: Couldn't find get_maintainer.pl"
>>          return []
>>
>> -    stdout = command.Output(get_maintainer, '--norolestats', fname)
>> +    stdout = command.Output(get_maintainer, '--norolestats', '--non', fname)
>>      return stdout.splitlines()
>> --
>> 1.9.1.423.g4596e3a
>>
>
> Good to avoid this problem, but I wonder if we should check the size
> and re-run the command if too long? That way we can keep names on the
> thread when it is possible.
>
> Also, why is there a limit on CC - is that a limitation described in
> the RFC, or just certain mailers? Can we get around it by specifying
> multiple Cc tags?

I believe that we're actually fighting a heuristic that the mailing
list servers have to avoid spammers.  In the past I've seen my mail
get through to list servers that weren't run by kernel.org but _not_
the the ones at kernel.org.  Kind of like how if you want to write
code that's for generic exynos 5 hardware you need to use exynos_5yyy
and _not_ replace the y with x (because that would be adult content!)

We could possibly only run the heuristic if the CC list was > 800
characters?  I'd hesitate to getting too close to 1024 since I think
there are instances where git will itself add a CC and that might blow
us out.

...another option is to just forget this patch and force people to add
this to ".get_maintainer.conf"

-Doug