Re: [PATCH v2] Add a b4 configuration file

[Cédric Le Goater]

On 1/2/25 23:53, Jiaxun Yang wrote:
> b4 [1] is a convenient tool to manage patch series with mailing list
> working flow.
>
> Add a project default config file to match QEMU's mailing list conventions
> as well as adopting differences on scripting.
>
> Examples of b4:
>
> ```
> $ b4 prep --check
> Checking patches using:
>    scripts/checkpatch.pl -q --terse --no-summary --mailback -
>
> ---
> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
> ---
> Changes in v2:
> - Add lore masks (philmd) from:
>    https://lore.kernel.org/qemu-devel/20241224135054.10243-1-philmd@linaro.org/
> - Link to v1: 
> https://lore.kernel.org/r/20241222-b4-config-v1-1-b3667beb30a4@flygoat.com
> ---
> ● cc5a4c890fed: Add a b4 configuration file
>    ● checkpatch.pl: 27: WARNING: added, moved or deleted file(s), does 
> MAINTAINERS need updating?
> ---
> Success: 0, Warning: 1, Error: 0
> ```
>
> ```
> $ b4 prep -c
> Will collect To: addresses using echo
> Will collect Cc: addresses using get_maintainer.pl
> Collecting To/Cc addresses
>      + To: qemu-devel@nongnu.org
> ---
> You can trim/expand this list with: b4 prep --edit-cover
> Invoking git-filter-repo to update the cover letter.
> New history written in 0.02 seconds...
> Completely finished after 0.06 seconds
> ```
>
> [1]: https://b4.docs.kernel.org/
>
> Co-developed-by: Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
> ---
>   .b4-config | 14 ++++++++++++++
>   1 file changed, 14 insertions(+)
>
> diff --git a/.b4-config b/.b4-config
> new file mode 100644
> index 
> 0000000000000000000000000000000000000000..4b9b2fe290f92f784cd3229616e2cbf0e8550e02
> --- /dev/null
> +++ b/.b4-config
> @@ -0,0 +1,14 @@
> +#
> +# Common b4 settings that can be used to send patches to QEMU upstream.
> +# https://b4.docs.kernel.org/
> +#
> +
> +[b4]
> +    send-series-to = qemu-devel@nongnu.org
> +    send-auto-to-cmd = echo
> +    send-auto-cc-cmd = scripts/get_maintainer.pl --noroles --norolestats 
> --nogit --nogit-fallback
> +    am-perpatch-check-cmd = scripts/checkpatch.pl -q --terse --no-summary 
> --mailback -
> +    prep-perpatch-check-cmd = scripts/checkpatch.pl -q --terse --no-summary 
> --mailback -
> +    searchmask = https://lore.kernel.org/qemu-devel/?x=m&t=1&q=%s
> +    linkmask = https://lore.kernel.org/qemu-devel/%s
> +    linktrailermask = Message-ID: <%s>

Why is this changing the link trailer ?

I think the default format :

  Link: https://lore.kernel.org/qemu-devel/<msgid>

is more convenient. people have a direct URL to click on.

Thanks,

C.



> ---
> base-commit: 65cb7129f4160c7e07a0da107f888ec73ae96776
> change-id: 20241222-b4-config-e469b075802c
>
> Best regards,