[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[dak/master] Remove commentsdir



Signed-off-by: Joerg Jaspert <joerg@debian.org>
---
 config/debian/dak.conf |    1 -
 dak/dakdb/update38.py  |   56 ++++++++++++++++++++++++++++++++++++++++++++++++
 dak/update_db.py       |    2 +-
 daklib/dbconn.py       |    1 -
 4 files changed, 57 insertions(+), 3 deletions(-)
 create mode 100755 dak/dakdb/update38.py

diff --git a/config/debian/dak.conf b/config/debian/dak.conf
index a4dd963..0d094c9 100644
--- a/config/debian/dak.conf
+++ b/config/debian/dak.conf
@@ -205,7 +205,6 @@ Suite
 	  contrib;
 	  non-free;
 	};
-	CommentsDir "/srv/ftp-master.debian.org/queue/p-u-new/COMMENTS/";
 	OverrideSuite "stable";
 	ValidTime 604800; // 7 days
 	VersionChecks
diff --git a/dak/dakdb/update38.py b/dak/dakdb/update38.py
new file mode 100755
index 0000000..583aa3a
--- /dev/null
+++ b/dak/dakdb/update38.py
@@ -0,0 +1,56 @@
+#!/usr/bin/env python
+# coding=utf8
+
+"""
+Remove more unused table columns
+
+@contact: Debian FTP Master <ftpmaster@debian.org>
+@copyright: 2010 Joerg Jaspert <joerg@debian.org>
+@license: GNU General Public License version 2 or later
+"""
+
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+
+################################################################################
+
+
+# <Ganneff> everyone - send your condolences to twerner, a new ftpmaster
+# <mhy> twerner: you poor, poor bastard
+# <mhy> twerner: look what it's done to me and Ganneff
+# <mhy> he used to be only 1.3m tall and I used to be female
+# <Tolimar> twerner: Congratulations... Uhm... Thorsta?
+# <Tolimar> Thorstine?
+# <DktrKranz> Thorstine seems the name of a candy we eat in Italy
+
+################################################################################
+
+import psycopg2
+from daklib.dak_exceptions import DBUpdateError
+
+################################################################################
+def do_update(self):
+    """
+    Remove unused table columns
+    """
+    print __doc__
+    try:
+        c = self.db.cursor()
+        c.execute('ALTER TABLE suite DROP COLUMN commentsdir')
+        c.execute("UPDATE config SET value = '38' WHERE name = 'db_revision'")
+        self.db.commit()
+
+    except psycopg2.ProgrammingError, msg:
+        self.db.rollback()
+        raise DBUpdateError, 'Unable to apply table-column update 38, rollback issued. Error message : %s' % (str(msg))
diff --git a/dak/update_db.py b/dak/update_db.py
index 1046607..11e5b1e 100755
--- a/dak/update_db.py
+++ b/dak/update_db.py
@@ -45,7 +45,7 @@ from daklib.dak_exceptions import DBUpdateError
 ################################################################################
 
 Cnf = None
-required_database_schema = 37
+required_database_schema = 38
 
 ################################################################################
 
diff --git a/daklib/dbconn.py b/daklib/dbconn.py
index 817afb2..f8c5376 100755
--- a/daklib/dbconn.py
+++ b/daklib/dbconn.py
@@ -2483,7 +2483,6 @@ SUITE_FIELDS = [ ('SuiteName', 'suite_name'),
                  ('Priority', 'priority'),
                  ('NotAutomatic', 'notautomatic'),
                  ('CopyChanges', 'copychanges'),
-                 ('CommentsDir', 'commentsdir'),
                  ('OverrideSuite', 'overridesuite')]
 
 class Suite(object):
-- 
1.5.6.5



Reply to: