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

Re: ok for git to delete temporary packs on write error?

From
DTDavid Tweed <david.tweed@gmail.com>
Date
Nov 28, 2007, 15:48 UTC
Message-ID
<e1dab3980711280748s53bf23f4j11be8c0472b7a216@mail.gmail.com>
In-Reply-To
<alpine.LFD.0.99999.0711281000590.9605@xanadu.home>
On Nov 28, 2007 3:01 PM, Nicolas Pitre <nico@cam.org> wrote:
Show 6 quoted lines
> On Wed, 28 Nov 2007, David Tweed wrote:
>
> > Hi, I'd like to check if there's any reason in the overall design of
> > git which would make deleting tmp_pack's that have suffered
> > write errors a bad idea? (Before I look further into this I may be missing
> > a good reason why they shouldn't be auto-deleted.)
[snip]
> I'd suggest they get removed a part of the prune command.

Mmm, I hadn't looked at the source and spotted the prune is actually a separate process to the repack. From a policy POV is it ok to assume any tmp_pack*'s existing when prune is run can be deleted? (I know you're not supposed to be having any other git ops going on whilst git gc works.)

Now I know they aren't being kept around for, say, "emergency error recovery" I may look to see if I can come up with an acceptable patch. (Don't hold your breath waiting.)

-- 
cheers, dave tweed__________________________
david.tweed@gmail.com
Rm 124, School of Systems Engineering, University of Reading.
"we had no idea that when we added templates we were adding a Turing-
complete compile-time language." -- C++ standardisation committee
Previous: Nicolas Pitre
Message 3 of 3 in “ok for git to delete temporary packs on write error?”
  1. David TweedNov 28, 2007
  2. Nicolas PitreNov 28, 2007
  3. David TweedNov 28, 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.