( )⚙ D11674 path: keep the path instance in the `pulloperation`

This is an archive of the discontinued Mercurial Phabricator instance.

Differential D11674

path: keep the path instance in the `pulloperation`
ClosedPublic

Authored by marmoute on Oct 15 2021, 4:18 AM.

Details

Reviewers
pulkit
Alphare
Group Reviewers
hg-reviewers
Commits
rHG7d1e60244561: path: keep the path instance in the `pulloperation`
Summary

This will allow more pull code to use the path options. Ideally we would modify
the peer API to keep the path instance. However that is much more churn that I
can deal with for my current goal: adjusting a user facing API for a new
feature before we release it in the 6.0 changesets. So I am taking a shortcut
that seems reasonable.

Diff Detail

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

Event Timeline

marmoute created this revision.Oct 15 2021, 4:18 AM
pulkit accepted this revision.Oct 15 2021, 8:53 AM
This revision is now accepted and ready to land.Oct 15 2021, 8:53 AM
marmoute updated this revision to Diff 30844.Oct 17 2021, 6:41 PM
Alphare accepted this revision.Oct 18 2021, 5:59 AM
marmoute added a commit: rHG7d1e60244561: path: keep the path instance in the `pulloperation`.Oct 18 2021, 7:53 AM
Closed by commit rHG7d1e60244561: path: keep the path instance in the `pulloperation` (authored by marmoute). · Explain Why
This revision was automatically updated to reflect the committed changes.
danchr added a subscriber: danchr.EditedOct 25 2021, 5:58 AM

In hg-git, we currently intercept mercurial.exchange.pull() and need to keep compatibility with both the upcoming release and earlier releases. The API change here — inserting a named argument _before_ a positional used by earlier releases — is rather obnoxious to work around. Sure, it's possible, but it won't be pretty…

Please consider moving the new argument to the end.

marmoute added a comment.Oct 29 2021, 8:19 AM
In D11674#179866, @danchr wrote:

In hg-git, we currently intercept mercurial.exchange.pull() and need to keep compatibility with both the upcoming release and earlier releases. The API change here — inserting a named argument _before_ a positional used by earlier releases — is rather obnoxious to work around. Sure, it's possible, but it won't be pretty…
Please consider moving the new argument to the end.

That seems reasonable, especially since we would hopefully merge it into the peer later.
Let me try to get a patch ready.

Revision Contents
Changeset List

PathPackages
Mmercurial/commands.py (1 line)
Mmercurial/exchange.py (11 lines)

Diff 30827

mercurial/commands.py

  • This file is larger than 256 KB, so syntax highlighting is disabled by default.
wlock = util.nullcontextmanager() wlock = util.nullcontextmanager()
if opts.get(b'update'): if opts.get(b'update'):
wlock = repo.wlock() wlock = repo.wlock()
with wlock: with wlock:
pullopargs.update(opts.get(b'opargs', {})) pullopargs.update(opts.get(b'opargs', {}))
modheads = exchange.pull( modheads = exchange.pull(
repo, repo,
other, other,
path=path,
heads=nodes, heads=nodes,
force=opts.get(b'force'), force=opts.get(b'force'),
bookmarks=opts.get(b'bookmark', ()), bookmarks=opts.get(b'bookmark', ()),
opargs=pullopargs, opargs=pullopargs,
confirm=opts.get(b'confirm'), confirm=opts.get(b'confirm'),
).cgresult ).cgresult
# brev is a name, which might be a bookmark to be activated at # brev is a name, which might be a bookmark to be activated at

mercurial/exchange.py

A new should be created at the beginning of each pull and discarded A new should be created at the beginning of each pull and discarded
afterward. afterward.
""" """
def __init__( def __init__(
self, self,
repo, repo,
remote, remote,
path,
heads=None, heads=None,
force=False, force=False,
bookmarks=(), bookmarks=(),
remotebookmarks=None, remotebookmarks=None,
streamclonerequested=None, streamclonerequested=None,
includepats=None, includepats=None,
excludepats=None, excludepats=None,
depth=None, depth=None,
): ):
# repo we pull into # repo we pull into
self.repo = repo self.repo = repo
# repo we pull from # repo we pull from
self.remote = remote self.remote = remote
# path object used to build this remote
#
# Ideally, the remote peer would carry that directly.
self.remote_path = path
# revision we try to pull (None is "all") # revision we try to pull (None is "all")
self.heads = heads self.heads = heads
# bookmark pulled explicitly # bookmark pulled explicitly
self.explicitbookmarks = [ self.explicitbookmarks = [
repo._bookmarks.expandname(bookmark) for bookmark in bookmarks repo._bookmarks.expandname(bookmark) for bookmark in bookmarks
] ]
# do we force pull? # do we force pull?
self.force = force self.force = force
raise error.Abort(b"user aborted") raise error.Abort(b"user aborted")
tr.addvalidator(b'900-pull-prompt', prompt) tr.addvalidator(b'900-pull-prompt', prompt)
def pull( def pull(
repo, repo,
remote, remote,
path,
heads=None, heads=None,
force=False, force=False,
bookmarks=(), bookmarks=(),
opargs=None, opargs=None,
streamclonerequested=None, streamclonerequested=None,
includepats=None, includepats=None,
excludepats=None, excludepats=None,
depth=None, depth=None,
includepats, excludepats = repo.narrowpats includepats, excludepats = repo.narrowpats
narrowspec.validatepatterns(includepats) narrowspec.validatepatterns(includepats)
narrowspec.validatepatterns(excludepats) narrowspec.validatepatterns(excludepats)
pullop = pulloperation( pullop = pulloperation(
repo, repo,
remote, remote,
heads, path=path,
force, heads=heads,
force=force,
bookmarks=bookmarks, bookmarks=bookmarks,
streamclonerequested=streamclonerequested, streamclonerequested=streamclonerequested,
includepats=includepats, includepats=includepats,
excludepats=excludepats, excludepats=excludepats,
depth=depth, depth=depth,
**pycompat.strkwargs(opargs) **pycompat.strkwargs(opargs)
) )