D3753: graft: store the node from where we started the operation in graftstate

yuja (Yuya Nishihara) phabricator at mercurial-scm.org
Sat Jun 16 10:04:20 EDT 2018


yuja added a comment.


  > - a/mercurial/commands.py +++ b/mercurial/commands.py @@ -2233,6 +2233,8 @@ raise error.Abort(_('no revisions specified')) cmdutil.checkunfinished(repo) cmdutil.bailifchanged(repo) +        # the node from where we started the graft operation +        statedata['startnode'] = repo['.'].node()
  
  Nit: Isn't it always the parent of the `newnodes[0]`?
  
  I don't have strong opinion whether we should store the startnode, but if
  we store it at the start of the graft, I think we won't need to care for
  `statedata.get('newnodes') is None` case, in the first patch. I mean we can
  always record newnodes no matter if it may be incomplete or not since the
  existence of the startnode gives us a trust.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D3753

To: pulkit, #hg-reviewers
Cc: yuja, mercurial-devel


More information about the Mercurial-devel mailing list