# HG changeset patch
# User Christophe de Vienne <christophe@unlish.com>
# Date 1412590121 -7200
# Node ID 335fa90d5ef4935300eedda49f8d1bb6ebaed59a
# Parent  c5637b5337e89e339dda0258c3776442413a196d
[devtools][pg] Remove a wrong assertion in _backup_database

When using pre_setup_database, a repo gets initialised before calling
backup_database. Hence self._repo is NOT None it that case.

diff -r c5637b5337e8 -r 335fa90d5ef4 devtools/__init__.py
--- a/devtools/__init__.py	Wed Nov 26 16:24:28 2014 +0100
+++ b/devtools/__init__.py	Mon Oct 06 12:08:41 2014 +0200
@@ -674,12 +674,14 @@
             backup_name = self._backup_name(db_id)
             self._drop(backup_name)
             self.system_source['db-name'] = backup_name
-            # during postgres database initialization, there is no repo set here.
-            assert self._repo is None
-            #self._repo.turn_repo_off()
-            createdb(self.helper, self.system_source, self.dbcnx, self.cursor, template=orig_name)
-            self.dbcnx.commit()
-            #self._repo.turn_repo_on()
+            if self._repo:
+                self._repo.turn_repo_off()
+            try:
+                createdb(self.helper, self.system_source, self.dbcnx, self.cursor, template=orig_name)
+                self.dbcnx.commit()
+            finally:
+                if self._repo:
+                    self._repo.turn_repo_on()
             return backup_name
         finally:
             self.system_source['db-name'] = orig_name