Re: [PATCH 0/5] make interpret-trailers useful for parsing
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Aug 10, 2017, 19:43 UTC
- Message-ID
- <xmqqr2wj2opv.fsf@gitster.mtv.corp.google.com>
- In-Reply-To
- <CAGZ79kaNrqvqr3OVcwhxFEMoS2sK8wvfVE_pmxXTWYVB-OBDXQ@mail.gmail.com>
Stefan Beller <sbeller@google.com> writes:
Show 27 quoted lines
> On Thu, Aug 10, 2017 at 11:03 AM, Jeff King <peff@peff.net> wrote: >> On Thu, Aug 10, 2017 at 04:02:46AM -0400, Jeff King wrote: >> >>> On Wed, Aug 09, 2017 at 08:21:47AM -0400, Jeff King wrote: >>> >>> > This series teaches interpret-trailers to parse and output just the >>> > trailers. So now you can do: >>> > >>> > $ git log --format=%B -1 8d44797cc91231cd44955279040dc4a1ee0a797f | >>> > git interpret-trailers --parse >>> > Signed-off-by: Hartmut Henkel <henkel@vh-s.de> >>> > Helped-by: Stefan Beller <sbeller@google.com> >>> > Signed-off-by: Ralf Thielow <ralf.thielow@gmail.com> >>> > Acked-by: Matthias Rüster <matthias.ruester@gmail.com> >>> >>> And here's a v2 that addresses all of the comments except one: Stefan >>> suggested that --only-existing wasn't a great name. I agree, but I like >>> everything else less. >> >> Here's a v3 that takes care of that (renaming it to --only-input). >> >> It's otherwise the same as v2, but since the name-change ripples through >> the remaining patches, I wanted to get v3 in front of people sooner >> rather than later. >> > > Looks good,
Yeah, looks good to me too. Thanks for the "--only-input" discussion.