Re: [PATCH v2 0/7] making log --first-parent imply -m
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Aug 3, 2020, 17:21 UTC
- Message-ID
- <xmqqy2mvve65.fsf@gitster.c.googlers.com>
- In-Reply-To
- <87pn87smw3.fsf@osv.gnss.ru>
Sergey Organov <sorganov@gmail.com> writes:
Show 18 quoted lines
> Junio C Hamano <gitster@pobox.com> writes: > >> Sergey Organov <sorganov@gmail.com> writes: >> >>> Sounds great, I only hoped we can do it right now, with this new >>> --diff-merges option, maybe as a pre-requisite to the patches in >>> question, but Jeff said it's too late, dunno why. >> >> A follow-up patch or two to remove the "--diff-merges" option and >> add the "--diff-parents=(none|<number>|c|cc|all)" option on top of >> the jk/log-fp-implies-m topic BEFORE it graduates to 'master' is a >> possibility. >> >> But is it worth the delay? I dunno. > > I don't think it's worth the delay, provided yet another new > --diff-parents is to be implemented rather that using --diff-merges for > that.
I was responding to your "it's too late, dunno why", as you seemed not to want to waste an option "--diff-merges" that will become unused when "--diff-parents" come and also wanted it to happen right now. If you no longer want to see it happen right now, that's OK by me.