| pulkit |
| hg-reviewers |
We want to deprecate ui.expandpath and simplify the code before adding more
complexity in the form of [paths] entry pointing to multiple url. So we inline
the relevant bits.
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
| Path | Packages | |||
|---|---|---|---|---|
| M | mercurial/utils/urlutil.py (22 lines) |
| url = path(ui, None, rawloc=source).rawloc | url = path(ui, None, rawloc=source).rawloc | ||||
| except ValueError: | except ValueError: | ||||
| url = source | url = source | ||||
| return parseurl(url, default_branches) | return parseurl(url, 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""" | ||||
| url = ui.expandpath(source) | if source is None: | ||||
| path, branch = parseurl(url, default_branches) | if b'default' in ui.paths: | ||||
| return url, path, branch | url = ui.paths[b'default'].rawloc | ||||
| else: | |||||
| # XXX this is the historical default behavior, but that is not | |||||
| # great, consider breaking BC on this. | |||||
| url = b'default' | |||||
| else: | |||||
| if source in ui.paths: | |||||
| url = ui.paths[source].rawloc | |||||
| else: | |||||
| # Try to resolve as a local path or URI. | |||||
| try: | |||||
| # we pass the ui instance are warning might need to be issued | |||||
| url = path(ui, None, rawloc=source).rawloc | |||||
| except ValueError: | |||||
| url = source | |||||
| clone_path, branch = parseurl(url, default_branches) | |||||
| return url, clone_path, branch | |||||
| def parseurl(path, branches=None): | def parseurl(path, branches=None): | ||||
| '''parse url#branch, returning (url, (branch, branches))''' | '''parse url#branch, returning (url, (branch, branches))''' | ||||
| u = url(path) | u = url(path) | ||||
| branch = None | branch = None | ||||
| if u.fragment: | if u.fragment: | ||||
| branch = u.fragment | branch = u.fragment | ||||