From: Junio C Hamano <gitster@pobox.com>
Date: Sun, 21 Mar 2021 16:52:30 GMT
Subject: Re: [PATCH v2 2/2] [GSOC] interpret-trailer: easy parse trailer value
Message-ID: <xmqqwnu0bga9.fsf@gitster.g>
In-Reply-To: <CAP8UFD1Oo-eWmoV-1mh1M=pA8+yRUYHy1wg8NvN3bthmCHBfvQ@mail.gmail.com>

Christian Couder <christian.couder@gmail.com> writes:

>> "Signed-off-by:C O <Mister@email.com>" is often too
>> verbose and error-prone.
>
> The 'trailer.<token>.command' config option can already be used to
> help with that, for example:
>
> -------
> $ git config trailer.sign.key "Signed-off-by: "
> $ git config trailer.sign.ifexists replace
> $ git config trailer.sign.command "git log --author='\$ARG' -1
> --format='format:%aN <%aE>'"
> $ git interpret-trailers --trailer sign=Linus<<EOF
> subject
>
> body
> EOF
> subject
>
> body
>
> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
> -------
>
> So even without this patch, after your first patch that implements
> `git commit --trailer ...`, it should be easy to setup something less
> verbose and less error-prone.

It is nice that it makes the complexity of 2/2 unnecessary ;-)