Re: [PATCH v1] convert: display progress for filtered objects that have been delayed

From: Junio C Hamano <gitster@pobox.com>
Date: Oct 4, 2017, 11:55 UTC
Message-ID: <xmqqr2uj9lfk.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <1AD84BB7-5BCA-4982-B157-944890F796EE@gmail.com>

Lars Schneider <larsxschneider@gmail.com> writes:

>> The delayed progress API is being simplified so I'll probably do a
>> bit of evil merge while merging this to 'pu'.
>
> I just realized that this patch got lost :-(

Really? Isn't it 52f1d62e ("convert: display progress for filtered objects that have been delayed", 2017-08-20)? It already is in 'master' since about 3 weeks ago.

As this is merely a new "feature", I do not think it is reasonable to expect it to be merged down to 'maint'. In fact, I think your original patch <20170820154720.32259-1-larsxschneider@gmail.com> declared that its Base Ref was "master", and I queued the topic on top of the then-current 'master', so it is not mergeable to 'maint'.

It may be cherry-pickable, but I do not think it deserves to be. It is not a fix for a grave bug or anything like that, right?