| phillco | |
| indygreg |
| hg-reviewers |
This was done the same way in both functions, so let's let the single
caller do it.
| Lint Skipped |
| Unit Tests Skipped |
| Path | Packages | |||
|---|---|---|---|---|
| M | hgext/rebase.py (10 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 |
| def _concludenode(self, rev, p1, p2, editor, commitmsg=None): | def _concludenode(self, rev, p1, p2, editor, commitmsg=None): | ||||
| '''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: | |||||
| commitmsg = ctx.description() | |||||
| 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), | extrafn=_makeextrafn(self.extrafns), | ||||
| commitmsg=commitmsg, | commitmsg=commitmsg, | ||||
| editor=editor, | editor=editor, | ||||
| keepbranches=self.keepbranchesf, | keepbranches=self.keepbranchesf, | ||||
| date=self.date) | date=self.date) | ||||
| 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, extrafn, keepbranches, | ||||
| date, commitmsg=None): | 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 but also store useful information in extra. | ||||
| Return node of committed revision.''' | Return node of committed revision.''' | ||||
| if commitmsg is None: | |||||
| commitmsg = ctx.description() | |||||
| keepbranch = keepbranches and repo[p1].branch() != ctx.branch() | keepbranch = keepbranches and repo[p1].branch() != ctx.branch() | ||||
| extra = {'rebase_source': ctx.hex()} | extra = {'rebase_source': ctx.hex()} | ||||
| if extrafn: | if extrafn: | ||||
| extrafn(ctx, extra) | 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: | ||||
| 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, extrafn, keepbranches, date, | ||||
| commitmsg=None): | 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. | 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()) | ||||
| if commitmsg is None: | |||||
| commitmsg = ctx.description() | |||||
| keepbranch = keepbranches and repo[p1].branch() != ctx.branch() | keepbranch = keepbranches and repo[p1].branch() != ctx.branch() | ||||
| extra = {'rebase_source': ctx.hex()} | extra = {'rebase_source': ctx.hex()} | ||||
| if extrafn: | if extrafn: | ||||
| extrafn(ctx, extra) | 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: | ||||