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

Re: Auto detaching head options (update) (Re: Working copy revision and push pain)

From
Jeff King <peff@peff.net>
Date
Mar 31, 2008, 01:59 UTC
Message-ID
<20080331015937.GA32645@coredump.intra.peff.net>
In-Reply-To
<7vy77zslor.fsf@gitster.siamese.dyndns.org>
On Sun, Mar 30, 2008 at 06:53:56PM -0700, Junio C Hamano wrote:
Show 8 quoted lines
> While I too find the proposal attractive as a concept, I am sorry to say
> that it would not work even with the current set of Porcelain (fwiw, when
> the index-base extension was done, the approach based on HEAD reflog would
> not have worked either).  This is because 605fac8 (update HEAD reflog when
> branch pointed to by HEAD is directly modified, 2007-03-21) made gremlin
> updates to the current branch tip to also update the reflog of HEAD.
> 
> If we revert that commit, we may be able to.

Eww. I didn't know we were doing that. In fact, that was one of the cases I mentioned as "this would cause the reflog to get out of sync, but that is a good thing because it would trigger your safety valve." But obviously I was wrong about how we behave now.

Why is 605fac8 beneficial? What common workflow is updating the HEAD branch through its actual name? If it is manipulating the index and worktree at the same time, then it should be using the name HEAD. If it isn't, then shouldn't the HEAD and index get out of sync (exactly the thing Jan's proposal is meant to protect against)?

-Peff
Previous: Junio C HamanoNext: Jeff King
Message 19 of 32 in “Working copy revision and push pain”
  1. Jonathan WattMar 23, 2008
  2. Johannes SchindelinMar 23, 2008
  3. Jonathan WattMar 23, 2008
  4. Elijah NewrenMar 23, 2008
  5. Jonathan WattMar 23, 2008
  6. Elijah NewrenMar 23, 2008
  7. Johannes SchindelinMar 23, 2008
  8. Jonathan WattMar 23, 2008
  9. Johannes SchindelinMar 23, 2008
  10. Jonathan WattMar 23, 2008
  11. Johannes SchindelinMar 23, 2008
  12. Auto detaching head options (Re: Working copy revision and push pain)Jan Hudec, Mar 25, 2008
  13. Johannes SchindelinMar 25, 2008
  14. Jeff KingMar 25, 2008
  15. Junio C HamanoMar 26, 2008
  16. Auto detaching head options (update) (Re: Working copy revision and push pain)Jan Hudec, Mar 29, 2008
  17. Jeff KingMar 29, 2008
  18. Junio C HamanoMar 31, 2008
  19. Jeff KingMar 31, 2008
  20. Jeff KingMar 31, 2008
  21. Elijah NewrenMar 23, 2008
  22. Jonathan WattMar 23, 2008
  23. Johannes SchindelinMar 23, 2008
  24. Johan HerlandMar 23, 2008
  25. Jonathan WattMar 23, 2008
  26. Junio C HamanoMar 23, 2008
  27. Junio C HamanoMar 23, 2008
  28. Johannes SchindelinMar 23, 2008
  29. Johannes SchindelinMar 24, 2008
  30. Johan HerlandMar 24, 2008
  31. Johannes SchindelinMar 23, 2008
  32. Jonathan WattMar 23, 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.