FeatureRequest: Drop Cc: lines if tags were given

Albert ARIBAUD albert.aribaud at gmail.com
Mon Oct 7 03:55:33 EST 2013


Le 06/10/2013 16:55, Wolfram Sang a écrit :
>
>> I'm not sure I understand why a Tested-by: would warrant the removal
>> of a Cc:.
>
> To me, before I apply a patch to my trees, Cc: means "person got
> notified of this patch". A tag means a person reacted to the patch, so
> the information that she was notified is superfluous.

In all patches that were Tested-by: a given person, some might Cc: this 
person, and some might not; now, if you remove Cc: because Tested-by: is 
present and should be considered an implicit Cc:, then you create the 
false information that the person would have been Cc: on /all/ patches 
-- or, conversively, you lose the information of which patch the person 
was really Cc: of.

IMO, a Tested-by: is the indication that someone tested the patch, and a 
Cc: is the indication that someone was copy on th patch, and these two 
are independent. Any simplification based on assumptions such as "a 
tester was necessarily a Cc" should be performed by ad hoc filtering on 
the client side, not by altering patches on the patchwork side.

> Regards,
>
>     Wolfram

Amicalement,
-- 
Albert.


More information about the Patchwork mailing list