D6356: mdiff: prepare mdiff to be used for run-tests to replace unidiff

sangeet259 (Sangeet Kumar Mishra) phabricator at mercurial-scm.org
Wed May 15 04:00:23 EDT 2019

sangeet259 added a subscriber: durin42.
sangeet259 added a comment.

  In https://phab.mercurial-scm.org/D6356#92644, @pulkit wrote:
  > > Why do I need to split the patches into two ?
  > >  The reason to split what would have been a single patch into two is because
  > >  for the next patch to be able to use the mdiff during tests,
  > >  this revision has to be there in the system's mercurial installation.
  > I didn't get the last line. Can you explain more?
  > I think both can be folded in one and should be good.
  Well the next patch imports mercurial and then calls `mdff.new_diff` . If I combine those two patches then you can not see the change as while testing https://phab.mercurial-scm.org/D6359 , if there `mdiff` doesn't already have new_diff 
  it will always fall back to unified_diff as happened with @durin42  , while he was trying to see the alternate path as he trying to test https://phab.mercurial-scm.org/D5514 .
  What I am trying to do here is, once https://phab.mercurial-scm.org/D6356 is has added `new_diff` to `mdiff`, you can then have this revision installed in your system or wherever one tests this, and then check https://phab.mercurial-scm.org/D6359 to see the alternate path it follows.

  rHG Mercurial


To: sangeet259, #hg-reviewers
Cc: durin42, pulkit, martinvonz, mercurial-devel

More information about the Mercurial-devel mailing list