Thread (3 messages) 3 messages, 3 authors, 2021-02-25

Re: [PATCH 1/1] mergetools/vimdiff: add vimdiff1 merge tool variant

flat view

From: David Aguilar <hidden>
Date: 2021-02-23 18:56:29

On Tue, Feb 16, 2021 at 10:42 AM Junio C Hamano [off-list ref] wrote:
Seth House [off-list ref] writes:
quoted
This adds yet another vimdiff/gvimdiff variant and presents conflicts as
a two-way diff between 'LOCAL' and 'REMOTE'. 'MERGED' is not opened
which deviates from the norm so usage text is echoed as a Vim message on
startup that instructs the user with how to proceed and how to abort.

Vimdiff is well-suited to two-way diffs so this is an option for a more
simple, more streamlined conflict resolution. For example: it is
difficult to communicate differences across more than two files using
only syntax highlighting; default vimdiff commands to get and put
changes between buffers do not need the user to manually specify
a source or destination buffer when only using two buffers.

Like other merge tools that directly compare 'LOCAL' with 'REMOTE', this
tool will benefit when paired with the new `mergetool.hideResolved`
setting.

Signed-off-by: Seth House <redacted>
---
 mergetools/vimdiff | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)
Any vimdiff$n + "git mergetool" users care to comment?  Thanks.
Nicely done, thanks Seth.

Looks good to me.  FWIW,
Tested-by: David Aguilar <redacted>

cheers,

--
David
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help