comparison hgext/rebase.py @ 37036:0ab0d94ead72

rebase: pass in "extra" itself into conclude[memory]node()

We were passing in a function instead for no clear reason (probably
historical, but I haven't bothered looking).

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

37035:b7f5d03e1e54

        Return node of committed revision.'''
        repo = self.repo
        ctx = repo[rev]
        if commitmsg is None:
            commitmsg = ctx.description()
        if self.inmemory:
            newnode = concludememorynode(repo, ctx, p1, p2,
                wctx=self.wctx,
                extrafn=_makeextrafn(self.extrafns),
                commitmsg=commitmsg,
                editor=editor,
                keepbranches=self.keepbranchesf,
                date=self.date)
            mergemod.mergestate.clean(repo)
        else:
            newnode = concludenode(repo, ctx, p1, p2,
                extrafn=_makeextrafn(self.extrafns),
                commitmsg=commitmsg,
                editor=editor,
                keepbranches=self.keepbranchesf,
                date=self.date)

    raise error.Abort(_('unable to collapse on top of %d, there is more '
                       'than one external parent: %s') %
                     (max(destancestors),
                      ', '.join("%d" % p for p in sorted(parents))))

def concludememorynode(repo, ctx, p1, p2, wctx, editor, extrafn, keepbranches,
                       date, commitmsg):
    '''Commit the memory changes with parents p1 and p2. Reuse commit info from
    ctx but also store useful information in extra.
    Return node of committed revision.'''
    keepbranch = keepbranches and repo[p1].branch() != ctx.branch()
    extra = {'rebase_source': ctx.hex()}
    if extrafn:
        extrafn(ctx, extra)

    destphase = max(ctx.phase(), phases.draft)
    overrides = {('phases', 'new-commit'): destphase}
    if keepbranch:
        overrides[('ui', 'allowemptycommit')] = True

            extra=extra, user=ctx.user(), branch=branch, editor=editor)
        commitres = repo.commitctx(memctx)
        wctx.clean() # Might be reused
        return commitres

def concludenode(repo, ctx, p1, p2, editor, extrafn, keepbranches, date,
                 commitmsg):
    '''Commit the wd changes with parents p1 and p2. Reuse commit info from ctx
    but also store useful information in extra.
    Return node of committed revision.'''
    dsguard = util.nullcontextmanager()
    if not repo.ui.configbool('rebase', 'singletransaction'):
        dsguard = dirstateguard.dirstateguard(repo, 'rebase')
    with dsguard:
        repo.setparents(repo[p1].node(), repo[p2].node())
        keepbranch = keepbranches and repo[p1].branch() != ctx.branch()
        extra = {'rebase_source': ctx.hex()}
        if extrafn:
            extrafn(ctx, extra)

        destphase = max(ctx.phase(), phases.draft)
        overrides = {('phases', 'new-commit'): destphase}
        if keepbranch:
            overrides[('ui', 'allowemptycommit')] = True

37036:0ab0d94ead72

        Return node of committed revision.'''
        repo = self.repo
        ctx = repo[rev]
        if commitmsg is None:
            commitmsg = ctx.description()
        extrafn = _makeextrafn(self.extrafns)
        extra = {'rebase_source': ctx.hex()}
        if extrafn:
            extrafn(ctx, extra)
        if self.inmemory:
            newnode = concludememorynode(repo, ctx, p1, p2,
                wctx=self.wctx,
                extra=extra,
                commitmsg=commitmsg,
                editor=editor,
                keepbranches=self.keepbranchesf,
                date=self.date)
            mergemod.mergestate.clean(repo)
        else:
            newnode = concludenode(repo, ctx, p1, p2,
                extra=extra,
                commitmsg=commitmsg,
                editor=editor,
                keepbranches=self.keepbranchesf,
                date=self.date)

    raise error.Abort(_('unable to collapse on top of %d, there is more '
                       'than one external parent: %s') %
                     (max(destancestors),
                      ', '.join("%d" % p for p in sorted(parents))))

def concludememorynode(repo, ctx, p1, p2, wctx, editor, extra, keepbranches,
                       date, commitmsg):
    '''Commit the memory changes with parents p1 and p2. Reuse commit info from
    ctx.
    Return node of committed revision.'''
    keepbranch = keepbranches and repo[p1].branch() != ctx.branch()

    destphase = max(ctx.phase(), phases.draft)
    overrides = {('phases', 'new-commit'): destphase}
    if keepbranch:
        overrides[('ui', 'allowemptycommit')] = True

            extra=extra, user=ctx.user(), branch=branch, editor=editor)
        commitres = repo.commitctx(memctx)
        wctx.clean() # Might be reused
        return commitres

def concludenode(repo, ctx, p1, p2, editor, extra, keepbranches, date,
                 commitmsg):
    '''Commit the wd changes with parents p1 and p2. Reuse commit info from ctx.
    Return node of committed revision.'''
    dsguard = util.nullcontextmanager()
    if not repo.ui.configbool('rebase', 'singletransaction'):
        dsguard = dirstateguard.dirstateguard(repo, 'rebase')
    with dsguard:
        repo.setparents(repo[p1].node(), repo[p2].node())
        keepbranch = keepbranches and repo[p1].branch() != ctx.branch()

        destphase = max(ctx.phase(), phases.draft)
        overrides = {('phases', 'new-commit'): destphase}
        if keepbranch:
            overrides[('ui', 'allowemptycommit')] = True