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

Re: git-cvsexportcommit fails for huge commits

From
Jeff King <peff@peff.net>
Date
Dec 13, 2007, 04:17 UTC
Message-ID
<20071213041737.GA23624@coredump.intra.peff.net>
In-Reply-To
<46a038f90712121158n674a9044t75ef99473314457c@mail.gmail.com>
On Thu, Dec 13, 2007 at 08:58:33AM +1300, Martin Langhoff wrote:
> IOWs, the protocol *is* atomic, and this patch does make things
> slightly more brittle. Perhaps require an option to be set before we
> do this?

I started writing a patch to let the user specify the limit, but it was just too ugly. What user knows the right limit? We are probably better off just setting the limit at something high and reasonable (like 64K -- it would be nice to get feedback from Markus on what platform he is using and what is a reasonable value), or just using a tempfile with xargs, which should figure out the correct value.

-Peff
Previous: Martin LanghoffNext: Markus Klinik
Message 13 of 15 in “git-cvsexportcommit fails for huge commits”
  1. Markus KlinikDec 11, 2007
  2. Jeff KingDec 12, 2007
  3. Junio C HamanoDec 12, 2007
  4. Jeff KingDec 12, 2007
  5. Junio C HamanoDec 14, 2007
  6. Jeff KingDec 14, 2007
  7. Jeff KingDec 14, 2007
  8. Robin RosenbergDec 14, 2007
  9. Jeff KingDec 15, 2007
  10. Linus TorvaldsDec 12, 2007
  11. Jeff KingDec 12, 2007
  12. Martin LanghoffDec 12, 2007
  13. Jeff KingDec 13, 2007
  14. Markus KlinikDec 13, 2007
  15. Jeff KingDec 13, 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.