# HG changeset patch
# User Siddharth Agarwal <sid0@fb.com>
# Date 2012-12-28 22:46:58
# Node ID ae663cba9a8d9cb1ca7f54d436136ff407af7793
# Parent  c351759ab0a07dcd92d6312ab0a02e5bfaafde4a

cmdutil: make getgraphlogrevs return revs in descending order

diff --git a/mercurial/cmdutil.py b/mercurial/cmdutil.py
--- a/mercurial/cmdutil.py
+++ b/mercurial/cmdutil.py
@@ -1430,8 +1430,12 @@ def getgraphlogrevs(repo, pats, opts):
     # Default --rev value depends on --follow but --follow behaviour
     # depends on revisions resolved from --rev...
     follow = opts.get('follow') or opts.get('follow_first')
+    possiblyunsorted = False # whether revs might need sorting
     if opts.get('rev'):
         revs = scmutil.revrange(repo, opts['rev'])
+        # Don't sort here because _makegraphlogrevset might depend on the
+        # order of revs
+        possiblyunsorted = True
     else:
         if follow and len(repo) > 0:
             revs = repo.revs('reverse(:.)')
@@ -1441,6 +1445,8 @@ def getgraphlogrevs(repo, pats, opts):
     if not revs:
         return iter([]), None, None
     expr, filematcher = _makegraphlogrevset(repo, pats, opts, revs)
+    if possiblyunsorted:
+        revs.sort(reverse=True)
     if expr:
         matcher = revset.match(repo.ui, expr)
         revs = increasingrevs(repo, revs, matcher)