Re: [PATCH 0/3] interpret-trailers: add --where, --if-exists, --if-missing
- Date: Wed, 12 Jul 2017 16:47:08 +0200
- From: Christian Couder <christian.couder@xxxxxxxxx>
- Subject: Re: [PATCH 0/3] interpret-trailers: add --where, --if-exists, --if-missing
On Wed, Jul 12, 2017 at 3:46 PM, Paolo Bonzini <bonzini@xxxxxxx> wrote:
> These options are useful to experiment with "git interpret-trailers"
> without having to tinker with .gitconfig. It can also be useful in the
> oddball case where you want a different placement for the trailer.
> The case that stimulated the creation of the patches was configuring
> trailer.signed-off-by.where = end
> and then wanting "--where before" when a patch author forgets his
> Signed-off-by and provides it in a separate email.
Maybe you could have used the following to temporarily override the config:
git -c trailer.signed-off-by.where=before interpret-trailers ...
But it could be helpful and more straightforward to provide the
options you implemented.
I am not sure also if --where should override both "trailer.where" and
"trailer.<token>.where", or if should just override the former.