From: Junio C Hamano <gitster@pobox.com>
Date: Fri, 30 Oct 2015 16:52:20 GMT
Subject: Re: [PATCH 0/2] checkout: added two options to control progress output
Message-ID: <xmqqa8r0mh8b.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <20151029220519.GA466@sigill.intra.peff.net>

Jeff King <peff@peff.net> writes:

>> --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.