git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Adding Reviewed-by/Tested-by tags to other peoples commits

From
Junio C Hamano <gitster@pobox.com>
Date
Oct 11, 2008, 11:28 UTC
Message-ID
<7v4p3jl6a9.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<20081011074604.GA23883@sigill.intra.peff.net>
Jeff King <peff@peff.net> writes:
Show 12 quoted lines
> On Sat, Oct 11, 2008 at 07:37:04AM +0100, Alex Bennee wrote:
>
>> I've just tested/reviewed a patch of someone elses and I want to
>> forward it on the appropriate mailing list. I gather for Linux you
>> just add the appropriate tags to the commit. Does git offer a shortcut
>> for doing this or do you have to do a reset HEAD^ and re-commit with a
>> copy&pasted and modified commit message?
>
> Try "git commit --amend" to edit the commit message. There's no
> automatic way of adding acked-by or tested-by tags with git; most people
> who do those things often would probably configure their editor to make
> it easier.

These Distimmed-by: fields are cumulative in nature, so once recorded in a commit log message you cannot add new ones without amending, which means you cannot exchange such commits by pushing or pulling but by forwarding patches via e-mails. I thought people added these in their MUAs. IOW, I share your "would probably" with s/editor/MUA/.

Previous: Johannes SchindelinNext: Johannes Schindelin
Message 4 of 8 in “Adding Reviewed-by/Tested-by tags to other peoples commits”
  1. Alex BenneeOct 11, 2008
  2. Jeff KingOct 11, 2008
  3. Johannes SchindelinOct 11, 2008
  4. Junio C HamanoOct 11, 2008
  5. Johannes SchindelinOct 11, 2008
  6. Junio C HamanoOct 11, 2008
  7. Johannes SchindelinOct 11, 2008
  8. Junio C HamanoOct 11, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.