( )⚙ D8913 scmutil: introduce filterrequirements() to split reqs into wc and store ones

This is an archive of the discontinued Mercurial Phabricator instance.

Differential D8913

scmutil: introduce filterrequirements() to split reqs into wc and store ones
ClosedPublic

Authored by pulkit on Aug 7 2020, 8:45 AM.

Details

Reviewers
indygreg
Group Reviewers
hg-reviewers
Commits
rHG9a99ab8217bd: scmutil: introduce filterrequirements() to split reqs into wc and store ones
Summary

In upcoming patches where we try to implement requirements in store, we will
need a mechanism to split all requirements on some basis and decide which one
goes to .hg/requires and which one goes to .hg/store/requires.

This patch introduce a separate function for that. Filtering logic for now is
put under an if False:. In upcoming patches it will be removed.

Diff Detail

Repository
rHG Mercurial
Branch
default
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

pulkit created this revision.Aug 7 2020, 8:45 AM
indygreg accepted this revision.Aug 8 2020, 1:17 PM
indygreg added a subscriber: indygreg.

I'm going to approve but will hold off landing until the following patch solidifies a bit.

This revision is now accepted and ready to land.Aug 8 2020, 1:17 PM
indygreg requested changes to this revision.Aug 24 2020, 8:42 PM
indygreg added inline comments.
mercurial/scmutil.py
1492

Since the presence or lack thereof of the requires file is important, I think we should change this to if x is not None so an empty file is written out if appropriate. This should happen, of course. But semantically it is the more correct behavior.

This revision now requires changes to proceed.Aug 24 2020, 8:42 PM
pulkit updated this revision to Diff 22443.Aug 25 2020, 7:38 AM
pulkit marked an inline comment as done.Aug 25 2020, 7:49 AM
indygreg accepted this revision.Aug 25 2020, 9:12 PM
This revision is now accepted and ready to land.Aug 25 2020, 9:12 PM

Revision Contents
Changeset List

PathPackages
Mmercurial/scmutil.py (17 lines)

Diff 22327

mercurial/scmutil.py

# Adjust the dirstate copies # Adjust the dirstate copies
for dst, src in pycompat.iteritems(copies): for dst, src in pycompat.iteritems(copies):
if src not in newctx or dst in newctx or ds[dst] != b'a': if src not in newctx or dst in newctx or ds[dst] != b'a':
src = None src = None
ds.copy(src, dst) ds.copy(src, dst)
repo._quick_access_changeid_invalidate() repo._quick_access_changeid_invalidate()
def filterrequirements(requirements):
""" filters the requirements into two sets:
wcreq: requirements which should be written in .hg/requires
storereq: which should be written in .hg/store/requires
Returns (wcreq, storereq)
"""
return requirements, None
def writereporequirements(repo, requirements=None): def writereporequirements(repo, requirements=None):
""" writes requirements for the repo to .hg/requires """ """ writes requirements for the repo to .hg/requires """
if requirements: if requirements:
repo.requirements = requirements repo.requirements = requirements
writerequires(repo.vfs, repo.requirements) wcreq, storereq = filterrequirements(repo.requirements)
if wcreq:
writerequires(repo.vfs, wcreq)
if storereq:
writerequires(repo.svfs, storereq)
indygregUnsubmitted
Done

Since the presence or lack thereof of the requires file is important, I think we should change this to if x is not None so an empty file is written out if appropriate. This should happen, of course. But semantically it is the more correct behavior.

indygreg: Since the presence or lack thereof of the `requires` file is important, I think we should…
def writerequires(opener, requirements): def writerequires(opener, requirements):
with opener(b'requires', b'w', atomictemp=True) as fp: with opener(b'requires', b'w', atomictemp=True) as fp:
for r in sorted(requirements): for r in sorted(requirements):
fp.write(b"%s\n" % r) fp.write(b"%s\n" % r)