This is an archive of the discontinued Mercurial Phabricator instance.

Differential D10872

urlutil: byteify several localized messages
ClosedPublic

Authored by mharbison72 on Jun 12 2021, 12:56 AM.

Details

Reviewers
Alphare
Group Reviewers
hg-reviewers
Commits
rHG515014d72aa2: urlutil: byteify several localized messages
Summary

Flagged by pytype.

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
Mmercurial/utils/urlutil.py (16 lines)

Diff 28582

mercurial/utils/urlutil.py

""" """
if dest is None: if dest is None:
dests = [] dests = []
else: else:
dests = [dest] dests = [dest]
dests = list(get_push_paths(repo, ui, dests)) dests = list(get_push_paths(repo, ui, dests))
if len(dests) != 1: if len(dests) != 1:
if dest is None: if dest is None:
msg = _("default path points to %d urls while %s only supports one") msg = _(
b"default path points to %d urls while %s only supports one"
)
msg %= (len(dests), action) msg %= (len(dests), action)
else: else:
msg = _("path points to %d urls while %s only supports one: %s") msg = _(b"path points to %d urls while %s only supports one: %s")
msg %= (len(dests), action, dest) msg %= (len(dests), action, dest)
raise error.Abort(msg) raise error.Abort(msg)
return dests[0] return dests[0]
def get_unique_pull_path(action, repo, ui, source=None, default_branches=()): def get_unique_pull_path(action, repo, ui, source=None, default_branches=()):
"""return a unique `(path, branch)` or abort if multiple are found """return a unique `(path, branch)` or abort if multiple are found
# Try to resolve as a local path or URI. # Try to resolve as a local path or URI.
path = try_path(ui, source) path = try_path(ui, source)
if path is not None: if path is not None:
urls.append(path.rawloc) urls.append(path.rawloc)
else: else:
urls.append(source) urls.append(source)
if len(urls) != 1: if len(urls) != 1:
if source is None: if source is None:
msg = _("default path points to %d urls while %s only supports one") msg = _(
b"default path points to %d urls while %s only supports one"
)
msg %= (len(urls), action) msg %= (len(urls), action)
else: else:
msg = _("path points to %d urls while %s only supports one: %s") msg = _(b"path points to %d urls while %s only supports one: %s")
msg %= (len(urls), action, source) msg %= (len(urls), action, source)
raise error.Abort(msg) raise error.Abort(msg)
return parseurl(urls[0], default_branches) return parseurl(urls[0], default_branches)
def get_clone_path(ui, source, default_branches=()): def get_clone_path(ui, source, default_branches=()):
"""return the `(origsource, path, branch)` selected as clone source""" """return the `(origsource, path, branch)` selected as clone source"""
urls = [] urls = []
path = try_path(ui, source) path = try_path(ui, source)
if path is not None: if path is not None:
urls.append(path.rawloc) urls.append(path.rawloc)
else: else:
urls.append(source) urls.append(source)
if len(urls) != 1: if len(urls) != 1:
if source is None: if source is None:
msg = _( msg = _(
"default path points to %d urls while only one is supported" b"default path points to %d urls while only one is supported"
) )
msg %= len(urls) msg %= len(urls)
else: else:
msg = _("path points to %d urls while only one is supported: %s") msg = _(b"path points to %d urls while only one is supported: %s")
msg %= (len(urls), source) msg %= (len(urls), source)
raise error.Abort(msg) raise error.Abort(msg)
url = urls[0] url = urls[0]
clone_path, branch = parseurl(url, default_branches) clone_path, branch = parseurl(url, default_branches)
return url, clone_path, branch return url, clone_path, branch
def parseurl(path, branches=None): def parseurl(path, branches=None):