Re: [PATCH v4 0/4] interpret-trailers: add --where, --if-exists, --if-missing

From: Junio C Hamano <gitster@pobox.com>
Date: Aug 14, 2017, 17:57 UTC
Message-ID: <xmqq1soevxq4.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <0ac82014-4da9-9ef2-5da0-07996c8aa8fd@gnu.org>

Paolo Bonzini <bonzini@gnu.org> writes:

Show 24 quoted lines

> On 01/08/2017 11:03, Paolo Bonzini wrote:
>> From: Paolo Bonzini <pbonzini@redhat.com>
>> 
>> These options are useful to experiment with "git interpret-trailers"
>> without having to tinker with .gitconfig (Junio said git should ahve
>> done this first and only added configuration afterwards).  It can
>> be useful in the case where you want a different placement for the trailer,
>> or for scripts/aliases that don't want to rely on specific .gitconfig
>> settings.
>> 
>> Compared to v2, the main change is that option order on the command-line
>> is respected.  That is,
>> 
>> 	--trailer 'acked-by: foo' --where end --trailer 'signed-off-by: me'
>> 
>> will only apply where=end to the second trailer.  Likewise,
>> 
>> 	--where end --trailer 'signed-off-by: me' --no-where \
>> 	--trailer 'acked-by: foo'
>> 
>> will only apply it to the first, reverting to trailer.*.where for the
>> "acked-by" trailer.
>
> Junio, I see you haven't yet applied this v4 to origin/pu, did you miss it?

Thanks for pinging. Either it was not noticed by mistake or was deliberately ignored during the pre-release freeze, I do not remember.