comparison hgext/rebase.py @ 40865:4edd427f34c1

rebase: clarify that commits that become empty are skipped Our message just said that it created no changes, but we didn't explicitly say that that meant that the the commit was not rebased. It also wasn't clear why it created no changes, so this patch also clarifies that that was because the changes were already in the destination. Differential Revision: https://phab.mercurial-scm.org/D5395
author Martin von Zweigbergk <martinvonz@google.com>
date Fri, 07 Dec 2018 14:23:17 -0800
parents 09680349cc2d
children 6acbe86c6490
comparison
equal deleted inserted replaced
40864:09680349cc2d 40865:4edd427f34c1
571 if newnode is not None: 571 if newnode is not None:
572 self.state[rev] = repo[newnode].rev() 572 self.state[rev] = repo[newnode].rev()
573 ui.debug('rebased as %s\n' % short(newnode)) 573 ui.debug('rebased as %s\n' % short(newnode))
574 else: 574 else:
575 if not self.collapsef: 575 if not self.collapsef:
576 ui.warn(_('note: rebase of %s created no changes ' 576 ui.warn(_('note: not rebasing %s, its destination already '
577 'to commit\n') % desc) 577 'has all its changes\n') % desc)
578 self.skipped.add(rev) 578 self.skipped.add(rev)
579 self.state[rev] = p1 579 self.state[rev] = p1
580 ui.debug('next revision set to %d\n' % p1) 580 ui.debug('next revision set to %d\n' % p1)
581 else: 581 else:
582 ui.status(_('already rebased %s as %s\n') % 582 ui.status(_('already rebased %s as %s\n') %