comparison hgext/rebase.py @ 35332:03bec089e105

rebase: disable `inmemory` if the rebaseset contains the working copy

As described in the comment, rebasing the working copy parent with in-memory
merge, and then updating to the new commit, isn't much faster because of the
extra overhead of uppdating. Best to leave it off in that case.

This commit makes deploying in-memory merge via an extension easier, because
you can just set `inmemory=True` based on some config or probability, and this
will turn off the cases where it's not desired.

Differential Revision: https://phab.mercurial-scm.org/D1616

35331:773a9a06047c

            if retcode is not None:
                return retcode
        else:
            destmap = _definedestmap(ui, repo, destf, srcf, basef, revf,
                                     destspace=destspace,
                                     inmemory=opts['inmemory'])
            retcode = rbsrt._preparenewrebase(destmap)
            if retcode is not None:
                return retcode

        tr = None

                rbsrt._performrebase(tr)

        rbsrt._finishrebase()

def _definedestmap(ui, repo, destf=None, srcf=None, basef=None, revf=None,
                   destspace=None, inmemory=False):
    """use revisions argument to define destmap {srcrev: destrev}"""
    if revf is None:
        revf = []

    # destspace is here to work around issues with `hg pull --rebase` see

    if revf and basef:
        raise error.Abort(_('cannot specify both a revision and a base'))
    if revf and srcf:
        raise error.Abort(_('cannot specify both a revision and a source'))

    if not inmemory:
        cmdutil.checkunfinished(repo)
        cmdutil.bailifchanged(repo)

    if ui.configbool('commands', 'rebase.requiredest') and not destf:
        raise error.Abort(_('you must specify a destination'),

                                'ancestor of destination %s\n') % dest)
            else: # can it happen?
                ui.status(_('nothing to rebase from %s to %s\n') %
                          ('+'.join(str(repo[r]) for r in base), dest))
            return None

    if not destf:
        dest = repo[_destrebase(repo, rebaseset, destspace=destspace)]
        destf = str(dest)

35332:03bec089e105

            if retcode is not None:
                return retcode
        else:
            destmap = _definedestmap(ui, repo, destf, srcf, basef, revf,
                                     destspace=destspace,
                                     opts=opts)
            rbsrt.inmemory = opts['inmemory']
            retcode = rbsrt._preparenewrebase(destmap)
            if retcode is not None:
                return retcode

        tr = None

                rbsrt._performrebase(tr)

        rbsrt._finishrebase()

def _definedestmap(ui, repo, destf=None, srcf=None, basef=None, revf=None,
                   destspace=None, opts=None):
    """use revisions argument to define destmap {srcrev: destrev}"""
    if revf is None:
        revf = []

    # destspace is here to work around issues with `hg pull --rebase` see

    if revf and basef:
        raise error.Abort(_('cannot specify both a revision and a base'))
    if revf and srcf:
        raise error.Abort(_('cannot specify both a revision and a source'))

    if not opts['inmemory']:
        cmdutil.checkunfinished(repo)
        cmdutil.bailifchanged(repo)

    if ui.configbool('commands', 'rebase.requiredest') and not destf:
        raise error.Abort(_('you must specify a destination'),

                                'ancestor of destination %s\n') % dest)
            else: # can it happen?
                ui.status(_('nothing to rebase from %s to %s\n') %
                          ('+'.join(str(repo[r]) for r in base), dest))
            return None
    # If rebasing the working copy parent, force in-memory merge to be off.
    #
    # This is because the extra work of checking out the newly rebased commit
    # outweights the benefits of rebasing in-memory, and executing an extra
    # update command adds a bit of overhead, so better to just do it on disk. In
    # all other cases leave it on.
    #
    # Note that there are cases where this isn't true -- e.g., rebasing large
    # stacks that include the WCP. However, I'm not yet sure where the cutoff
    # is.
    rebasingwcp = repo['.'].rev() in rebaseset
    if opts['inmemory'] and rebasingwcp:
        opts['inmemory'] = False
        # Check these since we did not before.
        cmdutil.checkunfinished(repo)
        cmdutil.bailifchanged(repo)

    if not destf:
        dest = repo[_destrebase(repo, rebaseset, destspace=destspace)]
        destf = str(dest)