This is an archive of the discontinued Mercurial Phabricator instance.

Differential D6542

remotefilelog: handle copies in changesets in getrenamedfn() override
ClosedPublic

Authored by martinvonz on Jun 19 2019, 2:31 PM.

Details

Reviewers
pulkit
Group Reviewers
hg-reviewers
Commits
rHGe387cb22f6c0: remotefilelog: handle copies in changesets in getrenamedfn() override
Summary

E.g. the {file_copies} template keyword didn't work with copies in
changesets before this patch because remotefilelog overrides the
getrenamedfn() and didn't handle the changeset-centric case.

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.

Revision Contents
Changeset List

PathPackages
Mhgext/remotefilelog/__init__.py (2 lines)

Diff 15604

hgext/remotefilelog/__init__.py

if len(set(x[3] for x in pendingfilecommits)) > 1: if len(set(x[3] for x in pendingfilecommits)) > 1:
raise error.ProgrammingError( raise error.ProgrammingError(
'pending multiple integer revisions are not supported') 'pending multiple integer revisions are not supported')
del pendingfilecommits[:] del pendingfilecommits[:]
return node return node
extensions.wrapfunction(changelog.changelog, 'add', changelogadd) extensions.wrapfunction(changelog.changelog, 'add', changelogadd)
def getrenamedfn(orig, repo, endrev=None): def getrenamedfn(orig, repo, endrev=None):
if not isenabled(repo): if not isenabled(repo) or copies.usechangesetcentricalgo(repo):
return orig(repo, endrev) return orig(repo, endrev)
rcache = {} rcache = {}
def getrenamed(fn, rev): def getrenamed(fn, rev):
'''looks up all renames for a file (up to endrev) the first '''looks up all renames for a file (up to endrev) the first
time the file is given. It indexes on the changerev and only time the file is given. It indexes on the changerev and only
parses the manifest if linkrev != changerev. parses the manifest if linkrev != changerev.