"sunlin via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes:
> From: Lin Sun <lin.sun@xxxxxxx>
>
> Make the mergetool used with "meld" backend behave similarly to "vimdiff" by
> telling it to auto-merge non-conflicting parts and highlight the conflicting
> parts when `mergetool.meld.useAutoMerge` is configured with `true`, or `auto`
> for detecting the `--auto-merge` option automatically.
>
> Helped-by: Đoàn Trần Công Danh <congdanhqx@xxxxxxxxx>
> Helped-by: David Aguilar <davvid@xxxxxxxxx>
> Signed-off-by: Lin Sun <lin.sun@xxxxxxx>
> Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx>
> ---
> Enable auto-merge for meld to follow the vimdiff beharior
>
> Hi, the mergetool "meld" does NOT merge the no-conflict changes, while
> the mergetool "vimdiff" will merge the no-conflict changes and highlight
> the conflict parts. This patch will make the mergetool "meld" similar to
> "vimdiff", auto-merge the no-conflict changes, highlight conflict parts.
This version seems to revert the bool-or-str change that would
become useful with a bit more polishing, with docs and tests.
Intended?
It also seems to repeat many issues that have been pointed out
during the reviews of previous revisions. I'd rather not have to
repeat pointing them out all over again, obviously.
I have queued v16 (the one with the beginning of bool-or-str) with
minimum fixups for issues I pointed out in the review of that
revision queued on top of it as SQUASH??? commit. Please find it in
the 'seen' branch.
Also,
Subject: Support auto-merge for meld to follow the vim-diff behavior
needs fixing.
cf. https://git-scm.com/docs/SubmittingPatches#describe-changes
Two tricks to pick a good title are:
- Read a pageful or two of "git shortlog --no-merges" output to get
accustomed to the general pattern in the entire project. It
would become clear why titles with "area:" prefix help the
patches with them easier to locate.
- Read a pageful of "git shortlog --no-merges -- mergetools"
(i.e. the same but limited to the files you are touching) to see
how the changes that contributed over time to build the subsystem
are called, so that the new patches can fit in the pattern.
I think something along the lines of
Subject: [PATCH] mergetool/meld: optionally allow --auto-merge behaviour
or something would fit well.
Thanks.
