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

Re: [PATCH] git-am: Run git gc only once and not for every patch.

From
Junio C Hamano <gitster@pobox.com>
Date
Jan 5, 2008, 06:55 UTC
Message-ID
<7vve68ep78.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<7vmyrli73h.fsf@gitster.siamese.dyndns.org>
Junio C Hamano <gitster@pobox.com> writes:
Show 10 quoted lines
> ...
> I do not think moving "gc --auto" outside the loop hurts in
> practice because you are not likely to be rebasing a truly huge
> series every day, but cruft can accumulate during "git am" run
> and the "gc --auto" inside loop was meant to clean them up.  The
> idea was taken from importers that run repack every once in a
> while (e.g. cvsimport runs every 1k commits), but "gc --auto"
> was designed to be much more lightweight than a full repack and
> that was the reason it was placed in the loop without counting
> "every N commits".

Having said all that, I'll take your patch as-is except that I'd drop the later part of the commit log message that explains Wine's practice.

Previous: Junio C HamanoNext: Michael Stefaniuc
Message 8 of 9 in “git-am: Run git gc only once and not for every patch.”
  1. git-am: Run git gc only once and not for every patch.Michael Stefaniuc, Jan 4, 2008
  2. Nicolas PitreJan 4, 2008
  3. Michael StefaniucJan 4, 2008
  4. Nicolas PitreJan 4, 2008
  5. Junio C HamanoJan 4, 2008
  6. Michael StefaniucJan 4, 2008
  7. Junio C HamanoJan 4, 2008
  8. Junio C HamanoJan 5, 2008
  9. Michael StefaniucJan 5, 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.