Re: [PATCH 0/3] interpret-trailers: add --where, --if-exists, --if-missing
- From
Christian Couder <christian.couder@gmail.com>
- Date
- Jul 12, 2017, 14:47 UTC
- Message-ID
- <CAP8UFD0bwb+Zeqn8Xg24J_Z639NiT3Awjzstk6o_L-Q+Wh+c0g@mail.gmail.com>
- In-Reply-To
- <20170712134646.17179-1-bonzini@gnu.org>
On Wed, Jul 12, 2017 at 3:46 PM, Paolo Bonzini <bonzini@gnu.org> wrote:
Show 11 quoted lines
> > 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.