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

Re: [PATCH 0/2] checkout: added two options to control progress output

From
Junio C Hamano <gitster@pobox.com>
Date
Oct 30, 2015, 16:52 UTC
Message-ID
<xmqqa8r0mh8b.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<20151029220519.GA466@sigill.intra.peff.net>
Jeff King <peff@peff.net> writes:
Show 7 quoted lines
>> --progress-lf: print progress information using LFs instead of CRs
>
> I notice this is part of your patch 1, but it really seems orthogonal to
> checkout's --progress option. It should probably be a separate patch,
> and it probably needs some justification in the commit message (i.e.,
> explain not just _what_ you are doing in the patch, but _why_ it is
> useful).

Yes, and as I doubted its validity, I would really have preferred to see it done as a later step. The "we want to say --[no-]progress" one on the other hand looked a very sensible change, and it was sad to see it taken hostage by that -lf thing.

Thanks.
Previous: Jeff King
Message 5 of 5 in “checkout: added two options to control progress output”
  1. 0/2 checkout: added two options to control progress outputEdmundo Carmona, Oct 24, 2015
  2. Jeff KingOct 29, 2015
  3. Edmundo Carmona AntoranzOct 30, 2015
  4. Jeff KingOct 30, 2015
  5. Junio C HamanoOct 30, 2015

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.