| Alphare |
| hg-reviewers |
This was showing up in an operation I was doing today, and I'd like to
be able to get trace spans for it instead of just profiler samples.
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
| def parseconfig(ui, raw, action): | def parseconfig(ui, raw, action): | ||||
| """Parse sparse config file content. | """Parse sparse config file content. | ||||
| action is the command which is trigerring this read, can be narrow, sparse | action is the command which is trigerring this read, can be narrow, sparse | ||||
| Returns a tuple of includes, excludes, and profiles. | Returns a tuple of includes, excludes, and profiles. | ||||
| """ | """ | ||||
| with util.timedcm( | |||||
| 'sparse.parseconfig(ui, %d bytes, action=%s)', len(raw), action | |||||
| ): | |||||
| includes = set() | includes = set() | ||||
| excludes = set() | excludes = set() | ||||
| profiles = set() | profiles = set() | ||||
| current = None | current = None | ||||
| havesection = False | havesection = False | ||||
| for line in raw.split(b'\n'): | for line in raw.split(b'\n'): | ||||
| line = line.strip() | line = line.strip() | ||||
| if not line or line.startswith(b'#'): | if not line or line.startswith(b'#'): | ||||
| # empty or comment line, skip | # empty or comment line, skip | ||||
| continue | continue | ||||
| elif line.startswith(b'%include '): | elif line.startswith(b'%include '): | ||||
| line = line[9:].strip() | line = line[9:].strip() | ||||
| if line: | if line: | ||||
| profiles.add(line) | profiles.add(line) | ||||
| elif line == b'[include]': | elif line == b'[include]': | ||||
| if havesection and current != includes: | if havesection and current != includes: | ||||
| # TODO pass filename into this API so we can report it. | # TODO pass filename into this API so we can report it. | ||||
| raise error.Abort( | raise error.Abort( | ||||
| _( | _( | ||||
| b'%(action)s config cannot have includes ' | b'%(action)s config cannot have includes ' | ||||
| b'after excludes' | b'after excludes' | ||||
| ) | ) | ||||
| % {b'action': action} | % {b'action': action} | ||||
| ) | ) | ||||
| havesection = True | havesection = True | ||||
| current = includes | current = includes | ||||
| continue | continue | ||||
| elif line == b'[exclude]': | elif line == b'[exclude]': | ||||
| havesection = True | havesection = True | ||||
| current = excludes | current = excludes | ||||
| elif line: | elif line: | ||||
| if current is None: | if current is None: | ||||
| raise error.Abort( | raise error.Abort( | ||||
| _( | _( | ||||
| b'%(action)s config entry outside of ' | b'%(action)s config entry outside of ' | ||||
| b'section: %(line)s' | b'section: %(line)s' | ||||
| ) | ) | ||||
| % {b'action': action, b'line': line}, | % {b'action': action, b'line': line}, | ||||
| hint=_( | hint=_( | ||||
| b'add an [include] or [exclude] line ' | b'add an [include] or [exclude] line ' | ||||
| b'to declare the entry type' | b'to declare the entry type' | ||||
| ), | ), | ||||
| ) | ) | ||||
| if line.strip().startswith(b'/'): | if line.strip().startswith(b'/'): | ||||
| ui.warn( | ui.warn( | ||||
| _( | _( | ||||
| b'warning: %(action)s profile cannot use' | b'warning: %(action)s profile cannot use' | ||||
| b' paths starting with /, ignoring %(line)s\n' | b' paths starting with /, ignoring %(line)s\n' | ||||
| ) | ) | ||||
| % {b'action': action, b'line': line} | % {b'action': action, b'line': line} | ||||
| ) | ) | ||||
| continue | continue | ||||
| current.add(line) | current.add(line) | ||||
| return includes, excludes, profiles | return includes, excludes, profiles | ||||
| # Exists as separate function to facilitate monkeypatching. | # Exists as separate function to facilitate monkeypatching. | ||||
| def readprofile(repo, profile, changeid): | def readprofile(repo, profile, changeid): | ||||
| """Resolve the raw content of a sparse profile file.""" | """Resolve the raw content of a sparse profile file.""" | ||||
| # TODO add some kind of cache here because this incurs a manifest | # TODO add some kind of cache here because this incurs a manifest | ||||
| # resolve and can be slow. | # resolve and can be slow. | ||||
| return repo.filectx(profile, changeid=changeid).data() | return repo.filectx(profile, changeid=changeid).data() | ||||