Page MenuHomePhabricator

revlog: adjust rewrite_sidedata code to not delete existing revlog content
ClosedPublic

Authored by marmoute on May 2 2021, 9:29 AM.

Details

Summary

The "w+" file mode is deleting all the content of the opened file. Which is bad…

This is not caught by the test because the test only check for a full, initial
pull where not pre-existing content exists. So we need to fix our test coverage
here.

However they are another issue that prevent "incremental" pull to work here. See
next changeset for details.

Diff Detail

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

Event Timeline

marmoute created this revision.May 2 2021, 9:29 AM
baymax updated this revision to Diff 27312.May 3 2021, 6:07 AM

✅ refresh by Heptapod after a successful CI run (🐙 💚)

baymax updated this revision to Diff 27336.May 3 2021, 7:36 AM

✅ refresh by Heptapod after a successful CI run (🐙 💚)

baymax updated this revision to Diff 27462.May 3 2021, 9:01 PM

✅ refresh by Heptapod after a successful CI run (🐙 💚)

Alphare accepted this revision.May 7 2021, 8:50 AM
This revision is now accepted and ready to land.May 7 2021, 8:50 AM