Show More
@@ -356,7 +356,6 b' class SvnRemote(object):' | |||
|
356 | 356 | raise Exception( |
|
357 | 357 | "Path %s is not a valid Subversion repository." % repo_path) |
|
358 | 358 | |
|
359 | # TODO: johbo: URL checks ? | |
|
360 | 359 | import subprocess |
|
361 | 360 | rdump = subprocess.Popen( |
|
362 | 361 | ['svnrdump', 'dump', '--non-interactive', src_url], |
@@ -370,6 +369,7 b' class SvnRemote(object):' | |||
|
370 | 369 | rdump.wait() |
|
371 | 370 | load.wait() |
|
372 | 371 | |
|
372 | log.debug('Return process ended with code: %s', rdump.returncode) | |
|
373 | 373 | if rdump.returncode != 0: |
|
374 | 374 | errors = rdump.stderr.read() |
|
375 | 375 | log.error('svnrdump dump failed: statuscode %s: message: %s', |
@@ -377,9 +377,12 b' class SvnRemote(object):' | |||
|
377 | 377 | reason = 'UNKNOWN' |
|
378 | 378 | if 'svnrdump: E230001:' in errors: |
|
379 | 379 | reason = 'INVALID_CERTIFICATE' |
|
380 | ||
|
381 | if reason == 'UNKNOWN': | |
|
382 | reason = 'UNKNOWN:{}'.format(errors) | |
|
380 | 383 | raise Exception( |
|
381 |
'Failed to dump the remote repository from %s.' % |
|
|
382 | reason) | |
|
384 | 'Failed to dump the remote repository from %s. Reason:%s' % ( | |
|
385 | src_url, reason)) | |
|
383 | 386 | if load.returncode != 0: |
|
384 | 387 | raise Exception( |
|
385 | 388 | 'Failed to load the dump of remote repository from %s.' % |
General Comments 0
You need to be logged in to leave comments.
Login now