From: Joerg Jaspert <joerg@debian.org>
Date: Wed, 23 Mar 2011 21:25:26 +0000 (+0100)
Subject: Merge remote-tracking branch 'ansgar/fix-update-db' into merge
X-Git-Url: https://git.decadent.org.uk/gitweb/?a=commitdiff_plain;h=4f29e33412c9bf42023483ef6f6e8dcf2fbe0dc5;hp=4577b5c4be4e7899f092a6d4fbca6c9c0380c333;p=dak.git

Merge remote-tracking branch 'ansgar/fix-update-db' into merge

* ansgar/fix-update-db:
  Fix dak update-db

Signed-off-by: Joerg Jaspert <joerg@debian.org>
---

diff --git a/dak/update_db.py b/dak/update_db.py
index 424359a3..77d00976 100755
--- a/dak/update_db.py
+++ b/dak/update_db.py
@@ -123,7 +123,7 @@ Updates dak's database schema to the lastest version. You should disable crontab
 
         try:
             # Build a connect string
-            if cnf["DB::Service"]:
+            if cnf.has_key("DB::Service"):
                 connect_str = "service=%s" % cnf["DB::Service"]
             else:
                 connect_str = "dbname=%s"% (cnf["DB::Name"])
@@ -134,7 +134,7 @@ Updates dak's database schema to the lastest version. You should disable crontab
 
         except:
             print "FATAL: Failed connect to database"
-            pass
+            sys.exit(1)
 
         database_revision = int(self.get_db_rev())
         logger.log(['transaction id before update: %s' % self.get_transaction_id()])