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

Re: [PATCH] refs: cleanup directories when deleting packed ref

From
WCWill Chandler <wfc@wfchandler.org>
Date
May 8, 2021, 04:27 UTC
Message-ID
<YJYTQYk5q8g6HaIm@mini.wfchandler.org>
In-Reply-To
<YJXF4xA0GFx2/2v4@coredump.intra.peff.net>
On Fri, May 07, 2021 at 06:57:39PM -0400, Jeff King wrote:
Show 5 quoted lines
> Getting back to your patch, though, you are definitely fixing a problem
> with update-ref (which correctly realizes there is no loose ref to clean
> up, but forgets that we had to make a lockfile). And the solution you
> have looks correct. I think you just need to update the test to exercise
> it with "update-ref -d".

Thanks for the thorough analysis! Apologies for the confusion, I really appreciate your patience.

I'll get the test fixed and note that this is specific to 'update-ref'.
Previous: Jeff KingNext: Will Chandler
Message 5 of 12 in “refs: cleanup directories when deleting packed ref”
  1. refs: cleanup directories when deleting packed refWill Chandler, May 7, 2021
  2. Jeff KingMay 7, 2021
  3. Jeff KingMay 7, 2021
  4. Jeff KingMay 7, 2021
  5. Will ChandlerMay 8, 2021
  6. Will ChandlerMay 8, 2021
  7. Bagas SanjayaMay 8, 2021
  8. Junio C HamanoMay 8, 2021
  9. Will ChandlerMay 9, 2021
  10. Junio C HamanoMay 10, 2021
  11. Jeff KingMay 11, 2021
  12. Junio C HamanoMay 11, 2021

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.