( )⚙ D6542 remotefilelog: handle copies in changesets in getrenamedfn() override

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
Lint Skipped
Unit
Unit Tests Skipped

Revision Contents
Changeset List

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

Diff 15592

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.