D715: repair: preserve phase also when not using generaldelta (issue5678)

[indygreg (Gregory Szorc)]

indygreg accepted this revision as: indygreg.
indygreg added a comment.


  This seems reasonable to me. I'm not sure it is appropriate for stable though.
  
  The temporary bundles should definitely use the latest available version. But changing the backup bundles (which are persisted after the operation completes) is a BC change and therefore marginally appropriate for stable. I'd feel better if we limited the behavior change on stable to just the temporary bundles and made backup bundle changes on default.
  
  I could be convinced this is appropriate for stable as written if we intended to store the latest available bundle format in backup bundles all along. This does seem reasonable...

REPOSITORY
  rHG Mercurial

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

To: martinvonz, #hg-reviewers, indygreg
Cc: indygreg, mercurial-devel