| hg-reviewers |
We will need to re-use this logic for path:// so we first extract it into its own method.
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
| Path | Packages | |||
|---|---|---|---|---|
| M | mercurial/ui.py (26 lines) |
| self.name = name | self.name = name | ||||
| self.rawloc = rawloc | self.rawloc = rawloc | ||||
| self.loc = b'%s' % u | self.loc = b'%s' % u | ||||
| self._validate_path() | self._validate_path() | ||||
| _path, sub_opts = ui.configsuboptions(b'paths', b'*') | _path, sub_opts = ui.configsuboptions(b'paths', b'*') | ||||
| self._own_sub_opts = {} | |||||
| if suboptions is not None: | if suboptions is not None: | ||||
| self._own_sub_opts = suboptions.copy() | |||||
| sub_opts.update(suboptions) | sub_opts.update(suboptions) | ||||
| self._all_sub_opts = sub_opts.copy() | |||||
| # Now process the sub-options. If a sub-option is registered, its | self._apply_suboptions(ui, sub_opts) | ||||
| # attribute will always be present. The value will be None if there | |||||
| # was no valid sub-option. | |||||
| for suboption, (attr, func) in pycompat.iteritems(_pathsuboptions): | |||||
| if suboption not in sub_opts: | |||||
| setattr(self, attr, None) | |||||
| continue | |||||
| value = func(ui, self, sub_opts[suboption]) | |||||
| setattr(self, attr, value) | |||||
| def _validate_path(self): | def _validate_path(self): | ||||
| # When given a raw location but not a symbolic name, validate the | # When given a raw location but not a symbolic name, validate the | ||||
| # location is valid. | # location is valid. | ||||
| if ( | if ( | ||||
| not self.name | not self.name | ||||
| and not self.url.scheme | and not self.url.scheme | ||||
| and not self._isvalidlocalpath(self.loc) | and not self._isvalidlocalpath(self.loc) | ||||
| ): | ): | ||||
| raise ValueError( | raise ValueError( | ||||
| b'location is not a URL or path to a local ' | b'location is not a URL or path to a local ' | ||||
| b'repo: %s' % self.rawloc | b'repo: %s' % self.rawloc | ||||
| ) | ) | ||||
| def _apply_suboptions(self, ui, sub_options): | |||||
| # Now process the sub-options. If a sub-option is registered, its | |||||
| # attribute will always be present. The value will be None if there | |||||
| # was no valid sub-option. | |||||
| for suboption, (attr, func) in pycompat.iteritems(_pathsuboptions): | |||||
| if suboption not in sub_options: | |||||
| setattr(self, attr, None) | |||||
| continue | |||||
| value = func(ui, self, sub_options[suboption]) | |||||
| setattr(self, attr, value) | |||||
| def _isvalidlocalpath(self, path): | def _isvalidlocalpath(self, path): | ||||
| """Returns True if the given path is a potentially valid repository. | """Returns True if the given path is a potentially valid repository. | ||||
| This is its own function so that extensions can change the definition of | This is its own function so that extensions can change the definition of | ||||
| 'valid' in this case (like when pulling from a git repo into a hg | 'valid' in this case (like when pulling from a git repo into a hg | ||||
| one).""" | one).""" | ||||
| try: | try: | ||||
| return os.path.isdir(os.path.join(path, b'.hg')) | return os.path.isdir(os.path.join(path, b'.hg')) | ||||
| # Python 2 may return TypeError. Python 3, ValueError. | # Python 2 may return TypeError. Python 3, ValueError. | ||||