Re: [PATCH for-4.5] scripts/get_maintainer.pl: Correctly CC the maintainers

[Julien Grall <julien.grall@linaro.org>]

On 11/20/2014 04:30 PM, Ian Campbell wrote:
> On Thu, 2014-11-20 at 16:15 +0000, Julien Grall wrote:
>> Hi Ian,
>>
>> On 11/20/2014 04:08 PM, Ian Campbell wrote:
>>> On Tue, 2014-11-18 at 20:03 +0000, Julien Grall wrote:
>>>> By default, the script get_maintainer.pl will remove duplicates email as soon
>>>> as it appends the list of maintainers of a new file, and therefore override
>>>> the role of the developper.
>>>>
>>>> On complex patch (see [1]), this will result to ommitting randomly some
>>>> maintainers.
>>>>
>>>> This could be fixed
>>>
>>> Are you proposing an alternative/better fix here? or describing what
>>> this patch does?
>>
>> Describing what the patch does.
> 
> Then I think you meant "This is fixed" or "This patches fixes this
> by ...".

Ok. I will fix it.

> 
>> By drawbacks I meant, if there is another bug in the script then we may
>> end up to cc too many people. Honestly I don't believe it's the case.
> 
> Sure, lets assume not.

I will drop this drawbacks.

Regards,

-- 
Julien Grall