# [PATCH] docs/pretty-formats: mention commas in %(trailers) syntax

7 messages from 2017-12-08 to 2017-12-09. Participants: Jeff King, Eric Sunshine, Junio C Hamano, Taylor Blau, Jacob Keller.
Thread: https://gitlist.dev/t/47405

## Jeff King, 2017-12-08 05:16

Subject: [PATCH] docs/pretty-formats: mention commas in %(trailers) syntax
Message-ID: <20171208051636.GA24693@sigill.intra.peff.net>
URL: https://gitlist.dev/e/20171208051636.GA24693%40sigill.intra.peff.net

```
Commit 84ff053d47 (pretty.c: delimit "%(trailers)" arguments
with ",", 2017-10-01) switched the syntax of the trailers
placeholder, but forgot to update the documentation in
pretty-formats.txt.

There's need to mention the old syntax; it was never in a
released version of Git.

Signed-off-by: Jeff King <peff@peff.net>
---
This should go on top of tb/delimit-pretty-trailers-args-with-comma.

 Documentation/pretty-formats.txt | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/Documentation/pretty-formats.txt b/Documentation/pretty-formats.txt
index d433d50f81..e664c088a5 100644
--- a/Documentation/pretty-formats.txt
+++ b/Documentation/pretty-formats.txt
@@ -204,11 +204,13 @@ endif::git-rev-list[]
   than given and there are spaces on its left, use those spaces
 - '%><(<N>)', '%><|(<N>)': similar to '% <(<N>)', '%<|(<N>)'
   respectively, but padding both sides (i.e. the text is centered)
-- %(trailers): display the trailers of the body as interpreted by
-  linkgit:git-interpret-trailers[1]. If the `:only` option is given,
-  omit non-trailer lines from the trailer block.  If the `:unfold`
-  option is given, behave as if interpret-trailer's `--unfold` option
-  was given. E.g., `%(trailers:only:unfold)` to do both.
+- %(trailers[:options]): display the trailers of the body as interpreted
+  by linkgit:git-interpret-trailers[1]. The `trailers` string may be
+  followed by a colon and zero or more comma-separated options. If the
+  `only` option is given, omit non-trailer lines from the trailer block.
+  If the `unfold` option is given, behave as if interpret-trailer's
+  `--unfold` option was given.  E.g., `%(trailers:only,unfold)` to do
+  both.
 
 NOTE: Some placeholders may depend on other options given to the
 revision traversal engine. For example, the `%g*` reflog options will
-- 
2.15.1.659.g8bd2eae3ea

```

## Eric Sunshine, 2017-12-08 08:10

Subject: Re: [PATCH] docs/pretty-formats: mention commas in %(trailers) syntax
Message-ID: <CAPig+cTdFSZziq9Mi7HHa+UcG3V9AwvWy1EpFoCLfT_9Fi+eKQ@mail.gmail.com>
URL: https://gitlist.dev/e/CAPig%2BcTdFSZziq9Mi7HHa%2BUcG3V9AwvWy1EpFoCLfT_9Fi%2BeKQ%40mail.gmail.com
In-Reply-To: <20171208051636.GA24693@sigill.intra.peff.net>

```
On Fri, Dec 8, 2017 at 12:16 AM, Jeff King <peff@peff.net> wrote:
> Commit 84ff053d47 (pretty.c: delimit "%(trailers)" arguments
> with ",", 2017-10-01) switched the syntax of the trailers
> placeholder, but forgot to update the documentation in
> pretty-formats.txt.
>
> There's need to mention the old syntax; it was never in a

I suppose you mean: s/need/no need/

> released version of Git.
>
> Signed-off-by: Jeff King <peff@peff.net>

```

## Jeff King, 2017-12-08 08:13

Subject: Re: [PATCH] docs/pretty-formats: mention commas in %(trailers) syntax
Message-ID: <20171208081356.GA24991@sigill.intra.peff.net>
URL: https://gitlist.dev/e/20171208081356.GA24991%40sigill.intra.peff.net
In-Reply-To: <CAPig+cTdFSZziq9Mi7HHa+UcG3V9AwvWy1EpFoCLfT_9Fi+eKQ@mail.gmail.com>

```
On Fri, Dec 08, 2017 at 03:10:34AM -0500, Eric Sunshine wrote:

> On Fri, Dec 8, 2017 at 12:16 AM, Jeff King <peff@peff.net> wrote:
> > Commit 84ff053d47 (pretty.c: delimit "%(trailers)" arguments
> > with ",", 2017-10-01) switched the syntax of the trailers
> > placeholder, but forgot to update the documentation in
> > pretty-formats.txt.
> >
> > There's need to mention the old syntax; it was never in a
> 
> I suppose you mean: s/need/no need/

Yes, indeed. Thanks.

-Peff

```

## Junio C Hamano, 2017-12-08 16:59

Subject: Re: [PATCH] docs/pretty-formats: mention commas in %(trailers) syntax
Message-ID: <xmqqk1xx9mut.fsf@gitster.mtv.corp.google.com>
URL: https://gitlist.dev/e/xmqqk1xx9mut.fsf%40gitster.mtv.corp.google.com
In-Reply-To: <20171208051636.GA24693@sigill.intra.peff.net>

```
Jeff King <peff@peff.net> writes:

> Commit 84ff053d47 (pretty.c: delimit "%(trailers)" arguments
> with ",", 2017-10-01) switched the syntax of the trailers
> placeholder, but forgot to update the documentation in
> pretty-formats.txt.
>
> There's need to mention the old syntax; it was never in a
> released version of Git.

There's or There's no?

>
> Signed-off-by: Jeff King <peff@peff.net>
> ---
> This should go on top of tb/delimit-pretty-trailers-args-with-comma.
>
>  Documentation/pretty-formats.txt | 12 +++++++-----
>  1 file changed, 7 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/pretty-formats.txt b/Documentation/pretty-formats.txt
> index d433d50f81..e664c088a5 100644
> --- a/Documentation/pretty-formats.txt
> +++ b/Documentation/pretty-formats.txt
> @@ -204,11 +204,13 @@ endif::git-rev-list[]
>    than given and there are spaces on its left, use those spaces
>  - '%><(<N>)', '%><|(<N>)': similar to '% <(<N>)', '%<|(<N>)'
>    respectively, but padding both sides (i.e. the text is centered)
> -- %(trailers): display the trailers of the body as interpreted by
> -  linkgit:git-interpret-trailers[1]. If the `:only` option is given,
> -  omit non-trailer lines from the trailer block.  If the `:unfold`
> -  option is given, behave as if interpret-trailer's `--unfold` option
> -  was given. E.g., `%(trailers:only:unfold)` to do both.
> +- %(trailers[:options]): display the trailers of the body as interpreted
> +  by linkgit:git-interpret-trailers[1]. The `trailers` string may be
> +  followed by a colon and zero or more comma-separated options. If the
> +  `only` option is given, omit non-trailer lines from the trailer block.
> +  If the `unfold` option is given, behave as if interpret-trailer's
> +  `--unfold` option was given.  E.g., `%(trailers:only,unfold)` to do
> +  both.
>  
>  NOTE: Some placeholders may depend on other options given to the
>  revision traversal engine. For example, the `%g*` reflog options will

```

## Junio C Hamano, 2017-12-08 17:00

Subject: Re: [PATCH] docs/pretty-formats: mention commas in %(trailers) syntax
Message-ID: <xmqqfu8l9mtu.fsf@gitster.mtv.corp.google.com>
URL: https://gitlist.dev/e/xmqqfu8l9mtu.fsf%40gitster.mtv.corp.google.com
In-Reply-To: <20171208081356.GA24991@sigill.intra.peff.net>

```
Jeff King <peff@peff.net> writes:

> On Fri, Dec 08, 2017 at 03:10:34AM -0500, Eric Sunshine wrote:
>
>> On Fri, Dec 8, 2017 at 12:16 AM, Jeff King <peff@peff.net> wrote:
>> > Commit 84ff053d47 (pretty.c: delimit "%(trailers)" arguments
>> > with ",", 2017-10-01) switched the syntax of the trailers
>> > placeholder, but forgot to update the documentation in
>> > pretty-formats.txt.
>> >
>> > There's need to mention the old syntax; it was never in a
>> 
>> I suppose you mean: s/need/no need/
>
> Yes, indeed. Thanks.

Ah, I probably should switch to 'read-only' mode until I finish my
inbox.


```

## Taylor Blau, 2017-12-09 03:44

Subject: Re: [PATCH] docs/pretty-formats: mention commas in %(trailers) syntax
Message-ID: <20171209034436.GA43611@D-10-157-251-166.dhcp4.washington.edu>
URL: https://gitlist.dev/e/20171209034436.GA43611%40D-10-157-251-166.dhcp4.washington.edu
In-Reply-To: <20171208051636.GA24693@sigill.intra.peff.net>

```
On Fri, Dec 08, 2017 at 12:16:36AM -0500, Jeff King wrote:
> Commit 84ff053d47 (pretty.c: delimit "%(trailers)" arguments
> with ",", 2017-10-01) switched the syntax of the trailers
> placeholder, but forgot to update the documentation in
> pretty-formats.txt.
>
> There's need to mention the old syntax; it was never in a
> released version of Git.
>
> Signed-off-by: Jeff King <peff@peff.net>

My mistake, and thank you for giving this your attention.

```

## Jacob Keller, 2017-12-09 08:06

Subject: Re: [PATCH] docs/pretty-formats: mention commas in %(trailers) syntax
Message-ID: <CA+P7+xr6XhnrxSk-jR0HyGksFsjBFkBuEr-RR8U-nEFUpJNJvg@mail.gmail.com>
URL: https://gitlist.dev/e/CA%2BP7%2Bxr6XhnrxSk-jR0HyGksFsjBFkBuEr-RR8U-nEFUpJNJvg%40mail.gmail.com
In-Reply-To: <20171208051636.GA24693@sigill.intra.peff.net>

```
On Thu, Dec 7, 2017 at 9:16 PM, Jeff King <peff@peff.net> wrote:
> Commit 84ff053d47 (pretty.c: delimit "%(trailers)" arguments
> with ",", 2017-10-01) switched the syntax of the trailers
> placeholder, but forgot to update the documentation in
> pretty-formats.txt.
>
> There's need to mention the old syntax; it was never in a
> released version of Git.
>
> Signed-off-by: Jeff King <peff@peff.net>
> ---
> This should go on top of tb/delimit-pretty-trailers-args-with-comma.
>
>  Documentation/pretty-formats.txt | 12 +++++++-----
>  1 file changed, 7 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/pretty-formats.txt b/Documentation/pretty-formats.txt
> index d433d50f81..e664c088a5 100644
> --- a/Documentation/pretty-formats.txt
> +++ b/Documentation/pretty-formats.txt
> @@ -204,11 +204,13 @@ endif::git-rev-list[]
>    than given and there are spaces on its left, use those spaces
>  - '%><(<N>)', '%><|(<N>)': similar to '% <(<N>)', '%<|(<N>)'
>    respectively, but padding both sides (i.e. the text is centered)
> -- %(trailers): display the trailers of the body as interpreted by
> -  linkgit:git-interpret-trailers[1]. If the `:only` option is given,
> -  omit non-trailer lines from the trailer block.  If the `:unfold`
> -  option is given, behave as if interpret-trailer's `--unfold` option
> -  was given. E.g., `%(trailers:only:unfold)` to do both.
> +- %(trailers[:options]): display the trailers of the body as interpreted
> +  by linkgit:git-interpret-trailers[1]. The `trailers` string may be
> +  followed by a colon and zero or more comma-separated options. If the
> +  `only` option is given, omit non-trailer lines from the trailer block.
> +  If the `unfold` option is given, behave as if interpret-trailer's
> +  `--unfold` option was given.  E.g., `%(trailers:only,unfold)` to do
> +  both.
>
>  NOTE: Some placeholders may depend on other options given to the
>  revision traversal engine. For example, the `%g*` reflog options will
> --
> 2.15.1.659.g8bd2eae3ea

HAH. I was recently looking at this documentation and going to send a
patch. Looks good besides the other reviewers comments.

Thanks,
Jake

```