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

Re: [PATCH v3 1/4] fetch-pack: new --stdin option to read refs from stdin

From
Jeff King <peff@peff.net>
Date
Apr 2, 2012, 20:42 UTC
Message-ID
<20120402204220.GC26503@sigill.intra.peff.net>
In-Reply-To
<4F79C22C.9080702@gmx.net>
On Mon, Apr 02, 2012 at 05:13:48PM +0200, Ivan Todoroski wrote:
> +		else {
> +			/* read from stdin one ref per line, until EOF */
> +			struct strbuf line;
> +			strbuf_init(&line, 0);
A minor style nit, but we usually spell this:
  struct strbuf line = STRBUF_INIT;
Other than that, this version looks good to me. Thanks.
-Peff
Previous: Ivan TodoroskiNext: Ivan Todoroski
Message 3 of 9 in “Clone fails on a repo with too many heads/tags”
  1. 0/4 Clone fails on a repo with too many heads/tagsIvan Todoroski, Apr 2, 2012
  2. 1/4 fetch-pack: new --stdin option to read refs from stdinIvan Todoroski, Apr 2, 2012
  3. Jeff KingApr 2, 2012
  4. 2/4 remote-curl: send the refs to fetch-pack on stdinIvan Todoroski, Apr 2, 2012
  5. Junio C HamanoApr 2, 2012
  6. 3/4 fetch-pack: test cases for the new --stdin optionIvan Todoroski, Apr 2, 2012
  7. Junio C HamanoApr 2, 2012
  8. Ivan TodoroskiApr 3, 2012
  9. 4/4 remote-curl: main test case for the OS command line overflowIvan Todoroski, Apr 2, 2012

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.