"GOT", but the "O" is a cute, smiling pufferfish. Index | Thread | Search

From:
Stefan Sperling <stsp@stsp.name>
Subject:
Re: tog: abort compound command
To:
Mark Jamsek <mark@jamsek.com>
Cc:
gameoftrees@openbsd.org
Date:
Sat, 2 Jul 2022 15:30:59 +0200

Download raw body.

Thread
On Sat, Jul 02, 2022 at 10:54:52PM +1000, Mark Jamsek wrote:
> The diff also removes a couple trailing whitespace in tog.1

It looks like this did not carry through properly in your mailer.
At least got patch fails to apply the diff cleanly:

$ got patch /tmp/p
#  tog/tog.1
@@ -295,7 +305,7 @@ hunk failed to apply
G  tog/tog.c
got: patch failed to apply

The - and + line for whitespace fixes are identical in what I received.

Feel free to commit such obvious fixes directly, so they don't
get in the way of diffs that need review.

> @@ -295,7 +305,7 @@ Find the Nth previous line which matches the current s
>  Toggle display of whitespace-only changes.
>  .It Cm A
>  Change the diff algorithm.
> -Supported diff algorithms are Myers and Patience.
> +Supported diff algorithms are Myers and Patience.
>  This is a global setting which also affects the
>  .Cm blame
>  view.
> @@ -385,7 +395,7 @@ Find the Nth previous line which matches the current s
>  (default: 1).
>  .It Cm A
>  Change the diff algorithm.
> -Supported diff algorithms are Myers and Patience.
> +Supported diff algorithms are Myers and Patience.
>  This is a global setting which also affects the
>  .Cm diff
>  view.