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.
--- 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: