This is an archive of the discontinued Mercurial Phabricator instance.

Differential D9748

upgrade: explicitly test for revlog index
ClosedPublic

Authored by marmoute on Jan 13 2021, 7:10 AM.

Details

Reviewers
None
Group Reviewers
hg-reviewers
Commits
rHG8023991dc811: upgrade: explicitly test for revlog index
rHGb30b31d7b68f: upgrade: explicitly test for revlog index
Summary

We do not want to exclude .d we want to only include .i. We are about to add
more extension type (for the persistent nodemap: .n, .nd) so lets make the
list explicit instead.

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 13 2021, 7:10 AM
marmoute added a commit: rHGb30b31d7b68f: upgrade: explicitly test for revlog index.Jan 13 2021, 2:41 PM
This revision was not accepted when it landed; it landed in state Needs Review.
Closed by commit rHGb30b31d7b68f: upgrade: explicitly test for revlog index (authored by marmoute). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mmercurial/upgrade_utils/engine.py (2 lines)

Diff 24812

mercurial/upgrade_utils/engine.py

# value is revlog_object_from_srcrepo # value is revlog_object_from_srcrepo
manifests = {} manifests = {}
changelogs = {} changelogs = {}
filelogs = {} filelogs = {}
# Perform a pass to collect metadata. This validates we can open all # Perform a pass to collect metadata. This validates we can open all
# source files and allows a unified progress bar to be displayed. # source files and allows a unified progress bar to be displayed.
for unencoded, encoded, size in alldatafiles: for unencoded, encoded, size in alldatafiles:
if unencoded.endswith(b'.d'): if not unencoded.endswith(b'.i'):
continue continue
rl = _revlogfrompath(srcrepo, unencoded) rl = _revlogfrompath(srcrepo, unencoded)
info = rl.storageinfo( info = rl.storageinfo(
exclusivefiles=True, exclusivefiles=True,
revisionscount=True, revisionscount=True,
trackedsize=True, trackedsize=True,