threads / discuss / 57153

request: allow passing -X <strategy-option> to git checkout <path> to auto-solve merge conflicts

Subject: request: allow passing -X <strategy-option> to git checkout <path> to auto-solve merge conflicts

## tl;dr

6 messages between Dec 27, 2021 and Dec 29, 2021.

replies: 5people: 2as markdown or json

Andrey Butirsky· Dec 27, 2021, 12:52 UTC · lore
Hi, stumbling upon this again and again, so decided to write finally,

while in conflicting state, the only thing we can do to auto-pick one or another side of conflict is passing --ours/--theirs option to git-checkout: git checkout --ours/--theirs <path>

The problem is - it doesn't actually do a _merge_, i.e. you lose all non-conflicted changes.

There is no easy way to solve that currently without third-party tools.

This link illustrates it: https://stackoverflow.com/a/68498101/1063363

Proposal: Shell we add -X <strategy-option> to git checkout <path> to allow it do a merge and _actually solve_ merge conflicts? That would be in-pair with other commands taking the option already: git-merge, git-rebase, (etc.?)

Andrey Butirsky· Dec 28, 2021, 20:44 UTC · lore

Re: request: allow passing -X <strategy-option> to git checkout <path> to auto-solve merge conflicts

Thanks Erik, please post your further replies to the mailing list so others could see it also.

On a topic, I'm not familiar with Git code-base so don't know if it even possible in it's current architecture..

On 28.12.2021 20:32, Erik Cervin Edin wrote:
Show 25 quoted lines
> That's my answer =)
>
> I think adding a merge strategy option to checkout might be useful
>
> On Mon, Dec 27, 2021 at 4:49 PM Andrey Butirsky <butirsky@gmail.com> wrote:
>> Hi, stumbling upon this again and again, so decided to write finally,
>>
>> while in conflicting state, the only thing we can do to auto-pick one or
>> another side of conflict is passing --ours/--theirs option to git-checkout:
>> git checkout --ours/--theirs <path>
>>
>> The problem is - it doesn't actually do a _merge_, i.e. you lose all
>> non-conflicted changes.
>>
>> There is no easy way to solve that currently without third-party tools.
>>
>> This link illustrates it:
>> https://stackoverflow.com/a/68498101/1063363
>>
>> Proposal:
>> Shell we add -X <strategy-option> to git checkout <path> to allow it do
>> a merge and _actually solve_ merge conflicts?
>> That would be in-pair with other commands taking the option already:
>> git-merge, git-rebase, (etc.?)
>>
Erik Cervin Edin· Dec 28, 2021, 21:50 UTC · re: Andrey Butirsky · lore

Re: request: allow passing -X <strategy-option> to git checkout <path> to auto-solve merge conflicts

> On Tue, Dec 28, 2021 at 9:44 PM Andrey Butirsky <butirsky@gmail.com> wrote:
> Thanks Erik, please post your further replies to the mailing list so
> others could see it also.
Mea culpa
> On a topic,
> I'm not familiar with Git code-base so don't know if it even possible in
> it's current architecture..

It looks like builtin/checkout.c checkout_merged is responsible and calls ll-merge.c ll_merge

I think other commands that allow merging strategies may use other "merge drivers".

From commit a944af1d86e6171d68ed2a3aa67b1d68f00e1fe8
Show 7 quoted lines
> merge: teach -Xours/-Xtheirs to binary ll-merge driver
>
> The (discouraged) -Xours/-Xtheirs modes of merge are supposed to
> give a quick and dirty way to come up with a random mixture of
> cleanly merged parts and punted conflict resolution to take contents
> from one side in conflicting parts.  These options however were only
> passed down to the low level merge driver for text.

It looks possible. But perhaps the sentiment is that it's not adviceable?

Erik Cervin Edin· Dec 29, 2021, 12:13 UTC · re: Erik Cervin Edin · lore

Re: request: allow passing -X <strategy-option> to git checkout <path> to auto-solve merge conflicts

On a tangent, you can set up your own merge tools.
So in git config you can add something like:

[mergetool "both"] cmd = "sed -i -e '/^<<<<<<<$/d' -e '/^=======$/d' -e '/^>>>>>>>$/d' -- $MERGED"

and call it with git mergetool --tool=both
On Tue, Dec 28, 2021 at 10:50 PM Erik Cervin Edin <erik@cervined.in> wrote:
Show 30 quoted lines
>
> > On Tue, Dec 28, 2021 at 9:44 PM Andrey Butirsky <butirsky@gmail.com> wrote:
> > Thanks Erik, please post your further replies to the mailing list so
> > others could see it also.
>
> Mea culpa
>
> > On a topic,
> > I'm not familiar with Git code-base so don't know if it even possible in
> > it's current architecture..
>
> It looks like
> builtin/checkout.c checkout_merged
> is responsible and calls
> ll-merge.c ll_merge
>
> I think other commands that allow merging strategies may use other
> "merge drivers".
>
> From commit a944af1d86e6171d68ed2a3aa67b1d68f00e1fe8
> > merge: teach -Xours/-Xtheirs to binary ll-merge driver
> >
> > The (discouraged) -Xours/-Xtheirs modes of merge are supposed to
> > give a quick and dirty way to come up with a random mixture of
> > cleanly merged parts and punted conflict resolution to take contents
> > from one side in conflicting parts.  These options however were only
> > passed down to the low level merge driver for text.
>
> It looks possible.
> But perhaps the sentiment is that it's not adviceable?
Andrey Butirsky· Dec 29, 2021, 13:35 UTC · re: Erik Cervin Edin · lore

Re: request: allow passing -X <strategy-option> to git checkout <path> to auto-solve merge conflicts

This is exactly what I tried to avoid, since Git already seem has all the needed under the hood to prevent such disaster..

On 29.12.2021 15:13, Erik Cervin Edin wrote:
Show 8 quoted lines
> On a tangent, you can set up your own merge tools.
>
> So in git config you can add something like:
>
> [mergetool "both"]
> cmd = "sed -i -e '/^<<<<<<<$/d' -e '/^=======$/d' -e '/^>>>>>>>$/d' -- $MERGED"
>
> and call it with git mergetool --tool=both
Erik Cervin Edin· Dec 29, 2021, 14:51 UTC · re: Andrey Butirsky · lore

Re: request: allow passing -X <strategy-option> to git checkout <path> to auto-solve merge conflicts

On Wed, Dec 29, 2021 at 2:35 PM Andrey Butirsky <butirsky@gmail.com> wrote:
>
> This is exactly what I tried to avoid, since Git already seem has all
> the needed under the hood to prevent such disaster..

It'd be neat if you could specify -X ours/theirs/both But I'm not sure I would describe the current situation as disastrous

← back to recent threads