| hg-reviewers |
The --status option filters Differential Revisions by their status, like
"Closed", "Accetped" etc. It could be useful to skip certain revisions.
Run phabread --status accepted --status closed Dxx on existing diffs and
make sure filtering works.
| Lint Skipped |
| Unit Tests Skipped |
| r'date': r'%d 0' % commit[r'time'], | r'date': r'%d 0' % commit[r'time'], | ||||
| r'node': commit[r'rev'], | r'node': commit[r'rev'], | ||||
| r'user': r'%s <%s>' % (commit[r'author'], commit[r'authorEmail']), | r'user': r'%s <%s>' % (commit[r'author'], commit[r'authorEmail']), | ||||
| } | } | ||||
| if len(commit.get(r'parents', ())) >= 1: | if len(commit.get(r'parents', ())) >= 1: | ||||
| meta[r'parent'] = commit[r'parents'][0] | meta[r'parent'] = commit[r'parents'][0] | ||||
| return meta or {} | return meta or {} | ||||
| def readpatch(repo, params, write, stack=False): | def _normalizestatus(name): | ||||
| """normalize status name: remove spaces and convert to lower case""" | |||||
| return name.replace(' ', '').lower() | |||||
| def readpatch(repo, params, write, stack=False, status=None): | |||||
| """generate plain-text patch readable by 'hg import' | """generate plain-text patch readable by 'hg import' | ||||
| write is usually ui.write. params is passed to "differential.query". If | write is usually ui.write. params is passed to "differential.query". If | ||||
| stack is True, also write dependent patches. | stack is True, also write dependent patches. If status is not None, it's a | ||||
| list of strings and Differential Revision outside them will be skipped. | |||||
| """ | """ | ||||
| statusset = set(_normalizestatus(s) for s in status or ()) | |||||
| # Differential Revisions | # Differential Revisions | ||||
| drevs = querydrev(repo, params, stack) | drevs = querydrev(repo, params, stack) | ||||
| # Prefetch hg:meta property for all diffs | # Prefetch hg:meta property for all diffs | ||||
| diffids = sorted(set(max(int(v) for v in drev[r'diffs']) for drev in drevs)) | diffids = sorted(set(max(int(v) for v in drev[r'diffs']) for drev in drevs)) | ||||
| diffs = callconduit(repo, 'differential.querydiffs', {'ids': diffids}) | diffs = callconduit(repo, 'differential.querydiffs', {'ids': diffids}) | ||||
| # Generate patch for each drev | # Generate patch for each drev | ||||
| for drev in drevs: | for drev in drevs: | ||||
| if statusset and _normalizestatus(drev[r'statusName']) not in statusset: | |||||
| continue | |||||
| repo.ui.note(_('reading D%s\n') % drev[r'id']) | repo.ui.note(_('reading D%s\n') % drev[r'id']) | ||||
| diffid = max(int(v) for v in drev[r'diffs']) | diffid = max(int(v) for v in drev[r'diffs']) | ||||
| body = callconduit(repo, 'differential.getrawdiff', {'diffID': diffid}) | body = callconduit(repo, 'differential.getrawdiff', {'diffID': diffid}) | ||||
| desc = getdescfromdrev(drev) | desc = getdescfromdrev(drev) | ||||
| header = '# HG changeset patch\n' | header = '# HG changeset patch\n' | ||||
| # Try to preserve metadata from hg:meta property. Write hg patch | # Try to preserve metadata from hg:meta property. Write hg patch | ||||
| # headers that can be read by the "import" command. See patchheadermap | # headers that can be read by the "import" command. See patchheadermap | ||||
| # and extract in mercurial/patch.py for supported headers. | # and extract in mercurial/patch.py for supported headers. | ||||
| meta = getdiffmeta(diffs[str(diffid)]) | meta = getdiffmeta(diffs[str(diffid)]) | ||||
| for k in _metanamemap.keys(): | for k in _metanamemap.keys(): | ||||
| if k in meta: | if k in meta: | ||||
| header += '# %s %s\n' % (_metanamemap[k], meta[k]) | header += '# %s %s\n' % (_metanamemap[k], meta[k]) | ||||
| write(('%s%s\n%s') % (header, desc, body)) | write(('%s%s\n%s') % (header, desc, body)) | ||||
| @command('phabread', | @command('phabread', | ||||
| [('', 'stack', False, _('read dependencies'))], | [('', 'stack', False, _('read dependencies')), | ||||
| ('', 'status', [], _('filter patches by status'))], | |||||
| _('REVID [OPTIONS]')) | _('REVID [OPTIONS]')) | ||||
| def phabread(ui, repo, revid, **opts): | def phabread(ui, repo, revid, **opts): | ||||
| """print patches from Phabricator suitable for importing | """print patches from Phabricator suitable for importing | ||||
| REVID could be a Differential Revision identity, like ``D123``, or just the | REVID could be a Differential Revision identity, like ``D123``, or just the | ||||
| number ``123``, or a full URL like ``https://phab.example.com/D123``. | number ``123``, or a full URL like ``https://phab.example.com/D123``. | ||||
| If --stack is given, follow dependencies information and read all patches. | If --stack is given, follow dependencies information and read all patches. | ||||
| If --status is given, only print Differential Revisions matching one of the | |||||
| given statuses. Statuses could be ``Accepted``, ``NeedsReview``, | |||||
| ``NeedsRevision``, ``Closed``, ``Abandoned``. | |||||
| """ | """ | ||||
| try: | try: | ||||
| revid = int(revid.split('/')[-1].replace('D', '')) | revid = int(revid.split('/')[-1].replace('D', '')) | ||||
| except ValueError: | except ValueError: | ||||
| raise error.Abort(_('invalid Revision ID: %s') % revid) | raise error.Abort(_('invalid Revision ID: %s') % revid) | ||||
| readpatch(repo, {'ids': [revid]}, ui.write, opts.get('stack')) | readpatch(repo, {'ids': [revid]}, ui.write, opts.get('stack'), | ||||
| opts.get('status')) | |||||