This is an archive of the discontinued Mercurial Phabricator instance.

rebase: use merge.graft() instead of merge.update()
Changes PlannedPublic

Authored by martinvonz on May 29 2020, 11:57 AM.

Details

Reviewers
marmoute
Group Reviewers
hg-reviewers
Summary

It's been possible to call merge.graft() here for a while (I don't
remember which of my previous changes made it possible), but it would
have meant that we did a redundant graftcopies() call. With the fix
to make merge.update() record copies in the overlayworkingctx (when
using that), we can now also remove the redundant graftcopies()
call. The next patch will do that.

Diff Detail

Repository
rHG Mercurial
Branch
default
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

martinvonz created this revision.May 29 2020, 11:57 AM

This patch seems related to D8596 which @martinvonz pointed as "not meant to be merged". What should we do with this diff ? is it intended for landing ? If not could get we it out of need-review and marke it RFC ?

This patch seems related to D8596 which @martinvonz pointed as "not meant to be merged". What should we do with this diff ? is it intended for landing ? If not could get we it out of need-review and marke it RFC ?

That was on an earlier version of the series; this series is now ready.

marmoute accepted this revision.Jun 10 2020, 1:54 PM

Great.

martinvonz abandoned this revision.Jul 16 2020, 11:23 AM

Changes planned to earlier patch in the series

martinvonz reclaimed this revision.Jul 16 2020, 11:24 AM
martinvonz planned changes to this revision.

Changes planned to earlier patch in the series