Page MenuHomePhabricator

requirements: no longer drop `generaldelta` requirement with revlogv2
Needs ReviewPublic

Authored by marmoute on Mon, May 3, 7:53 AM.

Details

Reviewers
None
Group Reviewers
hg-reviewers
Summary

A repository could use a mix of revlogv1 and revlogv2, making the requirements
still necessary. Overall we should move away from the "requirements" file being
used a way to configure the repository and stick to it "what do you need to
access this repository". However this is a wider work for another time.

In addition the logic we just dropped was confusing the hg debugformat
command, breaking the upgrade code and inconsistent (eg: sparse-revlog is also
implied by revlogv2).

Finally, multiple other config option would imply the use of the revlogv2
requirements, without drop the generaldelta one, leading to more
inconsistency.

Diff Detail

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

Event Timeline

marmoute created this revision.Mon, May 3, 7:53 AM
baymax updated this revision to Diff 27527.Mon, May 3, 9:44 PM

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

baymax updated this revision to Diff 27858.Tue, May 11, 1:42 PM

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