comparison mercurial/merge.py @ 23420:902554884335 stable 3.2.2

merge: before cd/dc prompt, check that changed side really changed

Before, merging would in some cases ask "wrong" questions about
"changed/deleted" conflicts ... and even do it before the resolve phase where
they can be postponed, re"resolved" or answered in bulk operations.

Instead, check that the content of the changed file really did change.

Reading and comparing file content is expensive and should be avoided before
the resolve phase. Prompting the user is however even more expensive. Checking
the content here is thus better.

The 'f in ancestors[0]' should not be necessary but is included to be extra
safe.

23419:a34a99181f36

            continue
        repo.ui.note(_('end of auction\n\n'))

    # Prompt and create actions. TODO: Move this towards resolve phase.
    for f, args, msg in actions['cd']:
        if repo.ui.promptchoice(
            _("local changed %s which remote deleted\n"
              "use (c)hanged version or (d)elete?"
              "$$ &Changed $$ &Delete") % f, 0):
            actions['r'].append((f, None, "prompt delete"))
        else:
            actions['a'].append((f, None, "prompt keep"))
    del actions['cd'][:]

    for f, args, msg in actions['dc']:
        flags, = args
        if repo.ui.promptchoice(
            _("remote changed %s which local deleted\n"
              "use (c)hanged version or leave (d)eleted?"
              "$$ &Changed $$ &Deleted") % f, 0) == 0:
            actions['g'].append((f, (flags,), "prompt recreating"))
    del actions['dc'][:]

23420:902554884335

            continue
        repo.ui.note(_('end of auction\n\n'))

    # Prompt and create actions. TODO: Move this towards resolve phase.
    for f, args, msg in actions['cd']:
        if f in ancestors[0] and not wctx[f].cmp(ancestors[0][f]):
            # local did change but ended up with same content
            actions['r'].append((f, None, "prompt same"))
        elif repo.ui.promptchoice(
            _("local changed %s which remote deleted\n"
              "use (c)hanged version or (d)elete?"
              "$$ &Changed $$ &Delete") % f, 0):
            actions['r'].append((f, None, "prompt delete"))
        else:
            actions['a'].append((f, None, "prompt keep"))
    del actions['cd'][:]

    for f, args, msg in actions['dc']:
        flags, = args
        if f in ancestors[0] and not mctx[f].cmp(ancestors[0][f]):
            # remote did change but ended up with same content
            pass # don't get = keep local deleted
        elif repo.ui.promptchoice(
            _("remote changed %s which local deleted\n"
              "use (c)hanged version or leave (d)eleted?"
              "$$ &Changed $$ &Deleted") % f, 0) == 0:
            actions['g'].append((f, (flags,), "prompt recreating"))
    del actions['dc'][:]