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

Re: What's cooking in git.git (Aug 2025, #01; Sun, 3)

From
Jeff King <peff@peff.net>
Date
Aug 5, 2025, 12:51 UTC
Message-ID
<20250805125120.GA1291837@coredump.intra.peff.net>
In-Reply-To
<xmqqectq4ne9.fsf@gitster.g>
On Mon, Aug 04, 2025 at 05:34:22PM -0700, Junio C Hamano wrote:
Show 19 quoted lines
> Jeff King <peff@peff.net> writes:
> 
> > I just left a rather lengthy review of the split-HEAD patch in v4. I
> > think hit has a few bugs that we need to address.
> >
> > I'm not sure if the right answer is to just go back to the v3 version
> > that simply rejected the racy HEAD update (since that's more or less
> > what happens now and nobody complains).
> >
> > If we do want to stick with the "silently skip the racy HEAD update"
> > strategy from v4, I left some fixes there. But I'd worry more about
> > maintainability and regressions in the future. I'm not sure if my hacky
> > "pretend the HEAD is this for splitting" patch is something we'd want to
> > carry or not. But if so, I think we could at least get a little coverage
> > in the test suite.
> 
> Between the "honestry admit we failed and reject" and "silently
> pretend nothing bad happened", I'd have to say that the former may
> be more preferrable, and I hope people would agree.

I think I don't think it's pretending that nothing bad happened, though. The user did not ask us to update HEAD, and we are able to fill their original request completely. So there really was nothing bad that happened.

To me the options are "try to do the best thing in a racy situation, even though it almost never happens and will be complex and somewhat untested" vs "do the simple and stupid thing that does not happen enough for anyone to care too much".

-Peff
Previous: Junio C HamanoNext: Junio C Hamano
Message 7 of 20 in “What's cooking in git.git (Aug 2025, #01; Sun, 3)”
  1. Junio C HamanoAug 4, 2025
  2. Patrick SteinhardtAug 4, 2025
  3. Junio C HamanoAug 4, 2025
  4. Patrick SteinhardtAug 4, 2025
  5. Jeff KingAug 4, 2025
  6. Junio C HamanoAug 5, 2025
  7. Jeff KingAug 5, 2025
  8. Junio C HamanoAug 5, 2025
  9. Jeff KingAug 5, 2025
  10. Patrick SteinhardtAug 5, 2025
  11. Jeff KingAug 5, 2025
  12. Junio C HamanoAug 5, 2025
  13. D. Ben KnobleAug 5, 2025
  14. Junio C HamanoAug 5, 2025
  15. D. Ben KnobleAug 5, 2025
  16. D. Ben KnobleAug 5, 2025
  17. Junio C HamanoAug 5, 2025
  18. Ben KnobleAug 5, 2025
  19. Lucas Seiki OshiroAug 6, 2025
  20. Junio C HamanoAug 6, 2025

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.