Re: [PATCH] sequencer.c: terminate the last line of author-script properly

From: Junio C Hamano <gitster@pobox.com>
Date: Jul 12, 2018, 20:23 UTC
Message-ID: <CAPc5daXRCtSSQTNxUz3DOL-sp=pgNKb13htmQaHR8KAjuUm3tg@mail.gmail.com>
In-Reply-To: <CAPig+cQWA+sE3iH89kQGWgS+0HQDK-V5++S0+F6_dpfOCfXxDg@mail.gmail.com>

Yup ;-)

On Thu, Jul 12, 2018 at 1:16 PM, Eric Sunshine <sunshine@sunshineco.com> wrote:

Show 15 quoted lines

> On Thu, Jul 12, 2018 at 4:13 PM Junio C Hamano <gitster@pobox.com> wrote:
>> I _think_ the right and safe way to fix taht is to do something like
>> this:
>>
>>         test -f .git/rebase-merge/author-script &&
>>         (
>>                 safe_unset GIT_AUTHOR_NAME GIT_AUTHOR_EMAIL ... &&
>
> s/safe_unset/sane_unset/
>
>>                 eval "$(cat .git/rebase-merge/author-script)" &&
>>                 test ... &&
>>                 test ... &&
>>                 test ...
>>         )