Re: [PATCH] interpret-trailers: obey scissors lines
- Date: Tue, 16 May 2017 15:42:05 +0900
- From: Junio C Hamano <gitster@xxxxxxxxx>
- Subject: Re: [PATCH] interpret-trailers: obey scissors lines
Brian Malehorn <bmalehorn@xxxxxxxxx> writes:
>> The mention of "'commit -v -s' works that way, too" was my attempt
>> to justify why it is OK to make this change unconditionally to
>> intepret-trailers, but I am still not 100% convinced with that
>> reasoning (or your original log message) that it is a safe thing to
>> do. It is not like its sole purpose is to serve as GIT_EDITOR for
>> the "git commit" command.
> Yup, this is a heuristic and it will sometimes hit false. But a far
> more grievous heuristic is ignoring all lines that begin with '#', yet
> we do that anyway for its convenience in editing commit messages. Since
> we already *partially* honor commit message syntax in
> interpret-trailers, we might as well *completely* honor it by parsing
> cut lines as well. At least, that's my reasoning.
I was primarily wondering if we want to protect this behind an
option to "interpret-trailers", instead of making this change
unconditionally. But your above reasoning makes sense to me.