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

[Kevin Hilman <khilman@baylibre.com>]

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.

Thanks for the fix!

Kevin

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