On 2/8/19 1:30 AM, Markus Armbruster wrote:
> Short story: please add
> 
>     [am]
>             messageid = true
> 
> to your .gitconfig.
> 
> Long story.  git-am can add a Message-id: tag.  Looks like this:
> 

> 
>         Signed-off-by: Thomas Huth <th...@redhat.com>
>         Reviewed-by: Daniel P. Berrangé <berra...@redhat.com>
>         Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com>
>         Tested-by: Philippe Mathieu-Daudé <phi...@redhat.com>
>         Acked-by: Alex Bennée <alex.ben...@linaro.org>
> --->    Message-id: 1549268743-18502-1-git-send-email-th...@redhat.com
>         Signed-off-by: Peter Maydell <peter.mayd...@linaro.org>
> 
> The Message-id identifies the patch e-mail.  It makes finding the review
> thread easier and more reliable.  It's also a valid key on Patchew[*].

I find the tag valuable enough in later git searches that I don't mind
feeding my own patches back through the mailing list to add it (patchew
helps with that, of course).  But for it to become mandatory, we'd need
to enhance scripts/checkpatch.pl to enforce it.

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org

Reply via email to