Page MenuHomePhabricator

legacy-revlog: fix requirement computation when cloning legacy repo
ClosedPublic

Authored by marmoute on Jan 12 2022, 9:09 AM.

Details

Summary

The oldest format of repository does not have requirements so we need to treat
them differently when cloning such repository.

The previous code had issue whenever we start using a working-copy-only
requirements. The "legacy" format is signaled using an empty requirements list.
If we add working-copy-only requirements to it, this is no longer empty, and
no longer legacy.

The new code fix this, and will get fully tested in a couple of changeset, once the
share-safe become the default.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

marmoute created this revision.Jan 12 2022, 9:09 AM
Alphare accepted this revision.Jan 17 2022, 8:47 AM
This revision is now accepted and ready to land.Jan 17 2022, 8:47 AM
marmoute updated this revision to Diff 31790.Jan 24 2022, 9:53 AM
baymax updated this revision to Diff 31804.Jan 24 2022, 10:01 AM

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

baymax updated this revision to Diff 31818.Jan 24 2022, 11:21 AM

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

baymax updated this revision to Diff 31863.Jan 24 2022, 1:13 PM

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