Page MenuHomePhabricator
Differential D10342

changegroup: don't limit cgv4 to revlogv2 repos
ClosedPublic
Actions

Authored by Alphare on Apr 9 2021, 4:49 AM.

Details

Reviewers
marmoute
Group Reviewers
hg-reviewers
Commits
rHG08e26ef4ad35: changegroup: don't limit cgv4 to revlogv2 repos
Summary

To help the transition from revlogv1 to revlogv2, we need to be able to enable
cgv4 for revlogv1 repos, so that revlogv2 clients can handle adding/removing
sidedata over the wire.

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

Alphare created this revision.Apr 9 2021, 4:49 AM
marmoute added a subscriber: marmoute.Apr 9 2021, 6:02 AM
marmoute added inline comments.
mercurial/changegroup.py
1696

Why is the config relevant here ?

(and if it is, this is not the right way to do it, since side-data can be implicitly enable by other options)

Alphare updated this revision to Diff 26777.Apr 10 2021, 3:14 PM
Alphare marked an inline comment as done.Apr 10 2021, 3:19 PM
Alphare updated this revision to Diff 27031.Apr 19 2021, 6:09 AM
Alphare updated this revision to Diff 27098.Apr 20 2021, 9:38 AM
marmoute added a comment.Apr 22 2021, 1:41 PM

So, the config will be, later, used to by the exchange code to forcily activate CG04 in situation that requires it ?

marmoute accepted this revision.Apr 22 2021, 1:56 PM
baymax updated this revision to Diff 27299.May 3 2021, 6:06 AM

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

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

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

Alphare added a commit: rHG08e26ef4ad35: changegroup: don't limit cgv4 to revlogv2 repos.May 5 2021, 6:39 PM
This revision was not accepted when it landed; it landed in state Needs Review.
Closed by commit rHG08e26ef4ad35: changegroup: don't limit cgv4 to revlogv2 repos (authored by Alphare). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mmercurial/changegroup.py (9 lines)
Mmercurial/configitems.py (5 lines)

Diff 27602

View Options

mercurial/changegroup.py

Loading...
View Options

mercurial/configitems.py

Loading...