Thread (1 message) 1 message, 1 author, 2016-06-15

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

From: Junio C Hamano <hidden>
Date: 2016-06-15 23:07:06

Jeff King [off-list ref] writes:
quoted
--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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help