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

Re: git push mirror mode

From
Andreas Ericsson <ae@op5.se>
Date
Nov 8, 2007, 13:48 UTC
Message-ID
<473313A6.40401@op5.se>
In-Reply-To
<20071108134432.GK9736@shadowen.org>
Andy Whitcroft wrote:
Show 30 quoted lines
> On Thu, Nov 08, 2007 at 02:14:12PM +0100, Andreas Ericsson wrote:
> 
>> Barring any errors in my understanding of the matter, here's how it
>> works for git.
>>
>> git separates author from committer, so code attribution is done with
>> author, and "I verified this is sane" is done by committer. Those two
>> usually only ever differ when the user tells git commit that the author
>> was someone else than him/her self, or when rewriting history with git
>> rebase or similar. git am also maintains authorship (using the From:
>> line in emails), but sets $committer to the person running it, so when
>> you apply patches sent by email from someone else you get the code
>> attribution right by default.
>>
>> The Signed-off-by line is, in git, used as "I touched the code here and
>> agree that it may be included in the mothership repo and all future
>> releases" (the spirit of that sentence is also in
>> Documentation/SubmittingPatches).
>>
>> We also have Acked-by (as does the kernel, no? I think we inherited it
>> from there) to mean something along the lines of "I vote we include this",
>> but not always based on technical merit (ie, patches can have many acks
>> without having ever been tested).
>>
>> Suggested-by, Tested-by and Reported-by are used less often, not always
>> written in dash-form, but hopefully always self-explanatory ;-)
> 
> What that doesn't tell me is how when sending an email carrying a patch
> one ensures the attribution is correct when loaded into git.
> 

Ach damn. I had a sentence there reading "From: can also be specified in the email body to attribute code to someone else than the sender." It's in my clipboard, but I forgot to paste it :-/

Show 6 quoted lines
> Having messed about with it a bit it does seem that if one wants git to
> attribute the patch to junio I have to add a From: line to the top of
> the email payload.
> 
> I'll resend so attributed.
> 
Thanks.
-- 
Andreas Ericsson                   andreas.ericsson@op5.se
OP5 AB                             www.op5.se
Tel: +46 8-230225                  Fax: +46 8-230231
Previous: Andy WhitcroftNext: Andy Whitcroft
Message 8 of 23 in “git push mirror mode”
  1. Andy WhitcroftNov 8, 2007
  2. 1/2 mirror pushingAndy Whitcroft, Nov 8, 2007
  3. 2/2 git-push: plumb in --mirror modeAndy Whitcroft, Nov 8, 2007
  4. Johannes SchindelinNov 8, 2007
  5. Andy WhitcroftNov 8, 2007
  6. Andreas EricssonNov 8, 2007
  7. Andy WhitcroftNov 8, 2007
  8. Andreas EricssonNov 8, 2007
  9. Andy WhitcroftNov 8, 2007
  10. Junio C HamanoNov 8, 2007
  11. 1/2 mirror pushingAndy Whitcroft, Nov 8, 2007
  12. Johannes SchindelinNov 8, 2007
  13. 2/2 git-push: plumb in --mirror modeAndy Whitcroft, Nov 8, 2007
  14. git push mirror mode V3Andy Whitcroft, Nov 8, 2007
  15. 4/4 tests: git push mirror mode testsAndy Whitcroft, Nov 8, 2007
  16. tests: git push mirror mode tests V2Andy Whitcroft, Nov 9, 2007
  17. tests: git push mirror mode tests V2 -- add tag testsAndy Whitcroft, Nov 9, 2007
  18. git-push: add documentation for the newly add --mirror modeAndy Whitcroft, Nov 9, 2007
  19. Junio C HamanoNov 9, 2007
  20. Andy WhitcroftNov 12, 2007
  21. 2/4 mirror pushing -- clean up match_refs flagsAndy Whitcroft, Nov 8, 2007
  22. 3/4 git-push: plumb in --mirror modeAndy Whitcroft, Nov 8, 2007
  23. 1/4 mirror pushingAndy Whitcroft, Nov 8, 2007

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.