# HG changeset patch # User Ian Moody # Date 1552084003 0 # Node ID 16d050678047133bffbf161565586ee81bf39ae3 # Parent 634b56b54e7c15fd623893fdd4f26097122ac0f0 py3: use r'' for group name arguments to MatchObjects in phabricator.py MatchObject group names are strings, not bytes. Using bytes in py3 leads to an IndexError. Differential Revision: https://phab.mercurial-scm.org/D6111 diff -r 634b56b54e7c -r 16d050678047 hgext/phabricator.py --- a/hgext/phabricator.py Sat Mar 09 01:58:51 2019 +0000 +++ b/hgext/phabricator.py Fri Mar 08 22:26:43 2019 +0000 @@ -304,7 +304,7 @@ # Check commit message m = _differentialrevisiondescre.search(ctx.description()) if m: - toconfirm[node] = (1, set(precnodes), int(m.group(b'id'))) + toconfirm[node] = (1, set(precnodes), int(m.group(r'id'))) # Double check if tags are genuine by collecting all old nodes from # Phabricator, and expect precursors overlap with it. @@ -555,7 +555,7 @@ # Create a local tag to note the association, if commit message # does not have it already m = _differentialrevisiondescre.search(ctx.description()) - if not m or int(m.group(b'id')) != newrevid: + if not m or int(m.group(r'id')) != newrevid: tagname = b'D%d' % newrevid tags.tag(repo, tagname, ctx.node(), message=None, user=None, date=None, local=True) @@ -1001,8 +1001,8 @@ m = _differentialrevisiondescre.search(ctx.description()) if m: return templateutil.hybriddict({ - b'url': m.group(b'url'), - b'id': b"D{}".format(m.group(b'id')), + b'url': m.group(r'url'), + b'id': b"D{}".format(m.group(r'id')), }) else: tags = ctx.repo().nodetags(ctx.node())