From patchwork Tue Feb 25 20:53:08 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: D8149: drawdag: abide by new createmarkers() API From: phabricator X-Patchwork-Id: 45317 Message-Id: To: Phabricator Cc: mercurial-devel@mercurial-scm.org Date: Tue, 25 Feb 2020 20:53:08 +0000 martinvonz created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY The `obsolete.createmarkers()` API was changed in 6335c0de80fa (obsolete: allow multiple predecessors in createmarkers, 2018-09-22) to prefer its precursors input to be a tuple instead of a single precursor. Let's fix `drawdag.py` to comply. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D8149 AFFECTED FILES tests/drawdag.py CHANGE DETAILS To: martinvonz, #hg-reviewers Cc: mercurial-devel diff --git a/tests/drawdag.py b/tests/drawdag.py --- a/tests/drawdag.py +++ b/tests/drawdag.py @@ -438,13 +438,13 @@ if cmd in (b'replace', b'rebase', b'amend'): nodes = [getctx(m) for m in arg.split(b'->')] for i in range(len(nodes) - 1): - rels.append((nodes[i], (nodes[i + 1],))) + rels.append(((nodes[i],), (nodes[i + 1],))) elif cmd in (b'split',): pre, succs = arg.split(b'->') succs = succs.split(b',') - rels.append((getctx(pre), (getctx(s) for s in succs))) + rels.append(((getctx(pre),), (getctx(s) for s in succs))) elif cmd in (b'prune',): for n in arg.split(b','): - rels.append((getctx(n), ())) + rels.append(((getctx(n),), ())) if rels: obsolete.createmarkers(repo, rels, date=(0, 0), operation=cmd)