From: ZheNing Hu <adlternative@gmail.com>
Date: Wed, 05 Apr 2023 07:46:45 GMT
Subject: Re: [PATCH v2 2/3] doc: interpret-trailers: don’t use deprecated config
Message-ID: <CAOLTT8QQABhiUsNiyDZj40yU2wu9RwkL=OANtLqzOrMioHMUZQ@mail.gmail.com>
In-Reply-To: <xmqqbkk44qbo.fsf@gitster.g>

Junio C Hamano <gitster@pobox.com> 于2023年4月4日周二 05:18写道：
>
> Kristoffer Haugsbakk <code@khaugsbakk.name> writes:
>
> > [2]: “At the same time, in order to allow `git interpret-trailers` to
> >    better simulate the behavior of `git command -s`,
> >    'trailer.<token>.cmd' will not automatically execute.”
>
> This may be a question more for Zhening than for you, but is the
> above a typo for "git commit -s", not "command"?
>

Ah, indeed it was a typo.

> > +$ git config trailer.see.cmd "glog-ref"
> > +$ cat msg.txt | git interpret-trailers --trailer=see
>
> This is inherited from the original, but it is a poor practice to
> run "cat" on a single file and pipe the result to another command.
> Just redirect from the file into the downstream command instead,
> i.e.
>
>     $ git interpret-trailers <empty-msg.txt
>
> Perhaps we should do that as a preliminary clean-up before these
> updates?