[PATCH 3 of 8] upgrade: rename `_copyrevlogs` to `_clonerevlogs`
Pierre-Yves David
pierre-yves.david at ens-lyon.org
Mon Aug 5 12:36:31 EDT 2019
# HG changeset patch
# User Pierre-Yves David <pierre-yves.david at octobus.net>
# Date 1564248347 -7200
# Sat Jul 27 19:25:47 2019 +0200
# Node ID aa19f478cfdfe781acc15cd26339644757156355
# Parent 085f8c53d0067f4cbb5ed3b9738ef33dbafa0782
# EXP-Topic upgrade-select
# Available At https://bitbucket.org/octobus/mercurial-devel/
# hg pull https://bitbucket.org/octobus/mercurial-devel/ -r aa19f478cfdf
upgrade: rename `_copyrevlogs` to `_clonerevlogs`
The underlying revlog method is named `clone`, keeping the naming consistent
seems clearer. This is motivated to clarify the difference with an (upcoming)
function that simply copy revlog files as is.
diff --git a/mercurial/upgrade.py b/mercurial/upgrade.py
--- a/mercurial/upgrade.py
+++ b/mercurial/upgrade.py
@@ -533,7 +533,7 @@ def _revlogfrompath(repo, path):
#reverse of "/".join(("data", path + ".i"))
return filelog.filelog(repo.svfs, path[5:-2])
-def _copyrevlogs(ui, srcrepo, dstrepo, tr, deltareuse, forcedeltabothparents):
+def _clonerevlogs(ui, srcrepo, dstrepo, tr, deltareuse, forcedeltabothparents):
"""Copy revlogs between 2 repos."""
revcount = 0
srcsize = 0
@@ -745,7 +745,7 @@ def _upgraderepo(ui, srcrepo, dstrepo, r
deltareuse = revlog.revlog.DELTAREUSEALWAYS
with dstrepo.transaction('upgrade') as tr:
- _copyrevlogs(ui, srcrepo, dstrepo, tr, deltareuse,
+ _clonerevlogs(ui, srcrepo, dstrepo, tr, deltareuse,
're-delta-multibase' in actions)
# Now copy other files in the store directory.
More information about the Mercurial-devel
mailing list