From 78fd5581a9fec142a1236eb90508724281f7f5ba Mon Sep 17 00:00:00 2001 From: Christopher Allan Webber Date: Mon, 10 Dec 2012 16:14:28 -0600 Subject: [PATCH] Remove print statments from this migration. The reason for wanting to give extra information to the user (this is a very special case migration) is good, but we don't have a nice "official" way to capture and present that information during tests, so removing this. --- mediagoblin/db/sql/migrations.py | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/mediagoblin/db/sql/migrations.py b/mediagoblin/db/sql/migrations.py index b4c91eee..99448bfa 100644 --- a/mediagoblin/db/sql/migrations.py +++ b/mediagoblin/db/sql/migrations.py @@ -181,10 +181,9 @@ def fix_CollectionItem_v0_constraint(db_conn): try: constraint.create() - except ProgrammingError as exc: - print "***", repr(exc) - print "***", repr(exc.statement) - print "***", repr(exc.params) - print "***", repr(exc.orig) - print "*** Ignoring it, you probably have a fresh install from a recent git." + except ProgrammingError: + # User probably has an install that was run since the + # collection tables were added, so we don't need to run this migration. + pass + db_conn.commit()