# HG changeset patch
# User Pierre-Yves David <pierre-yves.david@fb.com>
# Date 2014-11-01 22:59:37
# Node ID b6f7cf47f5d1ac1354d5686aa6edd2422e7fd9e4
# Parent  7361d8244efbd789fd199c7fd3fe831f1dd33d24

mq: do not call [0] on revset

The __getitem__ method have been removed. The "first" method is to be used
instead. Test have been extended to test this code path.

diff --git a/hgext/mq.py b/hgext/mq.py
--- a/hgext/mq.py
+++ b/hgext/mq.py
@@ -2001,7 +2001,7 @@ class queue(object):
                 base = repo.changelog.node(rev.first())
                 if base in [n.node for n in self.applied]:
                     raise util.Abort(_('revision %d is already managed')
-                                     % rev[0])
+                                     % rev.first())
                 if heads != [self.applied[-1].node]:
                     raise util.Abort(_('revision %d is not the parent of '
                                        'the queue') % rev.first())
diff --git a/tests/test-mq.t b/tests/test-mq.t
--- a/tests/test-mq.t
+++ b/tests/test-mq.t
@@ -1596,4 +1596,11 @@ Test interraction with revset (issue4426
   $ hg ci -m a
   $ hg qimport -r 0::
 
+reimport things
+
+  $ hg qimport -r 1::
+  abort: revision 2 is already managed
+  [255]
+
+
   $ cd ..