Submitter | Anton Shestakov |
---|---|
Date | Sept. 23, 2019, 3:39 a.m. |
Message ID | <d81e537a421f78d9f1bf.1569209989@df1> |
Download | mbox | patch |
Permalink | /patch/41731/ |
State | Accepted |
Headers | show |
Comments
On Mon, 23 Sep 2019 10:39:49 +0700, Anton Shestakov wrote: > # HG changeset patch > # User Anton Shestakov <av6@dwimlabs.net> > # Date 1569137636 -25200 > # Sun Sep 22 14:33:56 2019 +0700 > # Node ID d81e537a421f78d9f1bf6b3bdf736977ad126f68 > # Parent 44be33cf7a5720f681fe902c83d8cc4aa1c26127 > # EXP-Topic stack-object > stack: use repo.revs() instead of revsetlang.formatspec() + scmutil.revrange() Queued this patch, thanks. > diff --git a/mercurial/stack.py b/mercurial/stack.py > --- a/mercurial/stack.py > +++ b/mercurial/stack.py > @@ -8,8 +8,6 @@ > from __future__ import absolute_import > > from . import ( > - revsetlang, > - scmutil, > ) Dropped an empty import, which is syntactically invalid.
Patch
diff --git a/mercurial/stack.py b/mercurial/stack.py --- a/mercurial/stack.py +++ b/mercurial/stack.py @@ -8,8 +8,6 @@ from __future__ import absolute_import from . import ( - revsetlang, - scmutil, ) def getstack(repo, rev=None): @@ -23,7 +21,6 @@ def getstack(repo, rev=None): rev = '.' revspec = 'only(%s) and not public() and not ::merge()' - revset = revsetlang.formatspec(revspec, rev) - revisions = scmutil.revrange(repo, [revset]) + revisions = repo.revs(revspec, rev) revisions.sort() return revisions