| phillco | |
| indygreg |
| hg-reviewers |
We were passing in a function instead for no clear reason (probably
historical, but I haven't bothered looking).
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
| Path | Packages | |||
|---|---|---|---|---|
| M | hgext/rebase.py (23 lines) |
| Status | Author | Revision | |
|---|---|---|---|
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz |
| '''Commit the wd changes with parents p1 and p2. | '''Commit the wd changes with parents p1 and p2. | ||||
| Reuse commit info from rev but also store useful information in extra. | Reuse commit info from rev but also store useful information in extra. | ||||
| Return node of committed revision.''' | Return node of committed revision.''' | ||||
| repo = self.repo | repo = self.repo | ||||
| ctx = repo[rev] | ctx = repo[rev] | ||||
| if commitmsg is None: | if commitmsg is None: | ||||
| commitmsg = ctx.description() | commitmsg = ctx.description() | ||||
| extrafn = _makeextrafn(self.extrafns) | |||||
| extra = {'rebase_source': ctx.hex()} | |||||
| if extrafn: | |||||
| extrafn(ctx, extra) | |||||
| if self.inmemory: | if self.inmemory: | ||||
| newnode = concludememorynode(repo, ctx, p1, p2, | newnode = concludememorynode(repo, ctx, p1, p2, | ||||
| wctx=self.wctx, | wctx=self.wctx, | ||||
| extrafn=_makeextrafn(self.extrafns), | extra=extra, | ||||
| commitmsg=commitmsg, | commitmsg=commitmsg, | ||||
| editor=editor, | editor=editor, | ||||
| keepbranches=self.keepbranchesf, | keepbranches=self.keepbranchesf, | ||||
| date=self.date) | date=self.date) | ||||
| mergemod.mergestate.clean(repo) | mergemod.mergestate.clean(repo) | ||||
| else: | else: | ||||
| newnode = concludenode(repo, ctx, p1, p2, | newnode = concludenode(repo, ctx, p1, p2, | ||||
| extrafn=_makeextrafn(self.extrafns), | extra=extra, | ||||
| commitmsg=commitmsg, | commitmsg=commitmsg, | ||||
| editor=editor, | editor=editor, | ||||
| keepbranches=self.keepbranchesf, | keepbranches=self.keepbranchesf, | ||||
| date=self.date) | date=self.date) | ||||
| if newnode is None: | if newnode is None: | ||||
| # If it ended up being a no-op commit, then the normal | # If it ended up being a no-op commit, then the normal | ||||
| # merge state clean-up path doesn't happen, so do it | # merge state clean-up path doesn't happen, so do it | ||||
| return nullrev | return nullrev | ||||
| if len(parents) == 1: | if len(parents) == 1: | ||||
| return parents.pop() | return parents.pop() | ||||
| raise error.Abort(_('unable to collapse on top of %d, there is more ' | raise error.Abort(_('unable to collapse on top of %d, there is more ' | ||||
| 'than one external parent: %s') % | 'than one external parent: %s') % | ||||
| (max(destancestors), | (max(destancestors), | ||||
| ', '.join("%d" % p for p in sorted(parents)))) | ', '.join("%d" % p for p in sorted(parents)))) | ||||
| def concludememorynode(repo, ctx, p1, p2, wctx, editor, extrafn, keepbranches, | def concludememorynode(repo, ctx, p1, p2, wctx, editor, extra, keepbranches, | ||||
| date, commitmsg): | date, commitmsg): | ||||
| '''Commit the memory changes with parents p1 and p2. Reuse commit info from | '''Commit the memory changes with parents p1 and p2. Reuse commit info from | ||||
| ctx but also store useful information in extra. | ctx. | ||||
| Return node of committed revision.''' | Return node of committed revision.''' | ||||
| keepbranch = keepbranches and repo[p1].branch() != ctx.branch() | keepbranch = keepbranches and repo[p1].branch() != ctx.branch() | ||||
| extra = {'rebase_source': ctx.hex()} | |||||
| if extrafn: | |||||
| extrafn(ctx, extra) | |||||
| destphase = max(ctx.phase(), phases.draft) | destphase = max(ctx.phase(), phases.draft) | ||||
| overrides = {('phases', 'new-commit'): destphase} | overrides = {('phases', 'new-commit'): destphase} | ||||
| if keepbranch: | if keepbranch: | ||||
| overrides[('ui', 'allowemptycommit')] = True | overrides[('ui', 'allowemptycommit')] = True | ||||
| with repo.ui.configoverride(overrides, 'rebase'): | with repo.ui.configoverride(overrides, 'rebase'): | ||||
| # Replicates the empty check in ``repo.commit``. | # Replicates the empty check in ``repo.commit``. | ||||
| if wctx.isempty() and not repo.ui.configbool('ui', 'allowemptycommit'): | if wctx.isempty() and not repo.ui.configbool('ui', 'allowemptycommit'): | ||||
| branch = extra['branch'] | branch = extra['branch'] | ||||
| memctx = wctx.tomemctx(commitmsg, parents=(p1, p2), date=date, | memctx = wctx.tomemctx(commitmsg, parents=(p1, p2), date=date, | ||||
| extra=extra, user=ctx.user(), branch=branch, editor=editor) | extra=extra, user=ctx.user(), branch=branch, editor=editor) | ||||
| commitres = repo.commitctx(memctx) | commitres = repo.commitctx(memctx) | ||||
| wctx.clean() # Might be reused | wctx.clean() # Might be reused | ||||
| return commitres | return commitres | ||||
| def concludenode(repo, ctx, p1, p2, editor, extrafn, keepbranches, date, | def concludenode(repo, ctx, p1, p2, editor, extra, keepbranches, date, | ||||
| commitmsg): | commitmsg): | ||||
| '''Commit the wd changes with parents p1 and p2. Reuse commit info from ctx | '''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.''' | Return node of committed revision.''' | ||||
| dsguard = util.nullcontextmanager() | dsguard = util.nullcontextmanager() | ||||
| if not repo.ui.configbool('rebase', 'singletransaction'): | if not repo.ui.configbool('rebase', 'singletransaction'): | ||||
| dsguard = dirstateguard.dirstateguard(repo, 'rebase') | dsguard = dirstateguard.dirstateguard(repo, 'rebase') | ||||
| with dsguard: | with dsguard: | ||||
| repo.setparents(repo[p1].node(), repo[p2].node()) | repo.setparents(repo[p1].node(), repo[p2].node()) | ||||
| keepbranch = keepbranches and repo[p1].branch() != ctx.branch() | keepbranch = keepbranches and repo[p1].branch() != ctx.branch() | ||||
| extra = {'rebase_source': ctx.hex()} | |||||
| if extrafn: | |||||
| extrafn(ctx, extra) | |||||
| destphase = max(ctx.phase(), phases.draft) | destphase = max(ctx.phase(), phases.draft) | ||||
| overrides = {('phases', 'new-commit'): destphase} | overrides = {('phases', 'new-commit'): destphase} | ||||
| if keepbranch: | if keepbranch: | ||||
| overrides[('ui', 'allowemptycommit')] = True | overrides[('ui', 'allowemptycommit')] = True | ||||
| with repo.ui.configoverride(overrides, 'rebase'): | with repo.ui.configoverride(overrides, 'rebase'): | ||||
| # Commit might fail if unresolved files exist | # Commit might fail if unresolved files exist | ||||
| if date is None: | if date is None: | ||||