Re: [PATCH b4] Remove message-id from cc list

[Kevin Hilman <khilman@baylibre.com>]

Kevin Hilman <khilman@baylibre.com> writes:

> Louis Chauvet <louis.chauvet@bootlin.com> writes:
>
>> As the algorithm to detect mails to add in CC is very simple, it also
>> include the prerequisite-message-id to CC as they look like mails. In
>> order to avoid this kind of issue, filter out prerequisite-message-id
>> lines.
>>
>> Signed-off-by: Louis Chauvet <louis.chauvet@bootlin.com>
>> ---
>>  src/b4/__init__.py | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/src/b4/__init__.py b/src/b4/__init__.py
>> index ec230e7ace66..b0d91c0e4017 100644
>> --- a/src/b4/__init__.py
>> +++ b/src/b4/__init__.py
>> @@ -1990,7 +1990,7 @@ class LoreMessage:
>>  
>>      @staticmethod
>>      def find_trailers(body: str, followup: bool = False) -> Tuple[List[LoreTrailer], List[str]]:
>> -        ignores = {'phone', 'email'}
>> +        ignores = {'phone', 'email', 'prerequisite-message-id'}
>>          headers = {'subject', 'date', 'from'}
>>          links = {'link', 'buglink', 'closes'}
>>          nonperson = links | {'fixes', 'subject', 'date', 'obsoleted-by', 'change-id', 'base-commit'}
>>
>> ---
>> base-commit: 99b128995f27cd9aa6fa8adf7588f339919ac14d
>> change-id: 20240828-remove-invalid-cc-eecbd7d43b3d
>
> I have not been able to test (yet), but I'm guessing this should fix this
> bug[1] I reported a couple weeks ago.

I can now confirm that this patch fixes this bug[1].

Thanks Louis for the fix!

Kevin

P.S. is there some bugbot magic to connect this thread to an existing bug?

[1] https://bugzilla.kernel.org/show_bug.cgi?id=219138