# HG changeset patch
# User Thomas Arendsen Hein <thomas@intevation.de>
# Date 1186482265 -7200
# Node ID 0751c25ea0c0e7aedf3d17611b245a43d8bea43a
# Parent  07735e2d24cbfe5676514b0dd4ef8b74fdf1e1c5
raise util.Abort again if specified revision is not an integer.

This was changed to NoRepo in 462cc56d898f, because specifying non-integer
revisions for e.g. the Mercurial backend caused Abort to be raised in the
subversion importer.

Now util.Abort is raised again, but the check is done after verifying if it
really is a subversion repository.

diff --git a/hgext/convert/subversion.py b/hgext/convert/subversion.py
--- a/hgext/convert/subversion.py
+++ b/hgext/convert/subversion.py
@@ -108,11 +108,6 @@ class convert_svn(converter_source):
         self.lastrevs = {}
 
         latest = None
-        if rev:
-            try:
-                latest = int(rev)
-            except ValueError:
-                raise NoRepo('svn: revision %s is not an integer' % rev)
         try:
             # Support file://path@rev syntax. Useful e.g. to convert
             # deleted branches.
@@ -137,6 +132,12 @@ class convert_svn(converter_source):
         except SubversionException, e:
             raise NoRepo("couldn't open SVN repo %s" % self.url)
 
+        if rev:
+            try:
+                latest = int(rev)
+            except ValueError:
+                raise util.Abort('svn: revision %s is not an integer' % rev)
+
         try:
             self.get_blacklist()
         except IOError, e: