From: Joerg Jaspert <joerg@debian.org>
Date: Mon, 5 May 2008 14:16:03 +0000 (+0200)
Subject: Move the transitions exception into our own class too and use it
X-Git-Url: https://err.no/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=7e2d97048a4f48fb84843803dda8fdd710edda08;p=dak

Move the transitions exception into our own class too and use it
---

diff --git a/ChangeLog b/ChangeLog
index 869ff621..bdb58f18 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2008-05-05  Joerg Jaspert  <joerg@debian.org>
+
+	* daklib/dak_exceptions.py (dakerrors): add TransitionsError
+	* dak/transitions.py: Use it, instead of the own definition
+
 2008-05-05  Mark Hymers  <mhy@debian.org>
 
 	* daklib/dak_exceptions.py: Add a default message and tidy up our string
diff --git a/dak/transitions.py b/dak/transitions.py
index 7a0da2e1..b69c5732 100755
--- a/dak/transitions.py
+++ b/dak/transitions.py
@@ -27,6 +27,7 @@ import os, pg, sys, time, errno, fcntl, tempfile, pwd, re
 import apt_pkg
 import daklib.database as database
 import daklib.utils as utils
+from daklib.dak_extensions import TransitionsError
 import syck
 
 # Globals
@@ -227,9 +228,6 @@ def write_transitions(from_trans):
 
 ################################################################################
 
-class ParseException(Exception):
-    pass
-
 ##########################################
 #### This usually runs within sudo !! ####
 ##########################################
@@ -248,7 +246,7 @@ def write_transitions_from_file(from_file):
     else:
         trans = load_transitions(from_file)
         if trans is None:
-            raise ParseException, "Unparsable transitions file %s" % (file)
+            raise TransitionsError, "Unparsable transitions file %s" % (file)
         write_transitions(trans)
 
 ################################################################################
@@ -458,7 +456,7 @@ def main():
     if Options["import"]:
         try:
             write_transitions_from_file(Options["import"])
-        except ParseException, m:
+        except TransitionsError, m:
             print m
             sys.exit(2)
         sys.exit(0)
diff --git a/daklib/dak_exceptions.py b/daklib/dak_exceptions.py
index e488ac9d..b52e0657 100644
--- a/daklib/dak_exceptions.py
+++ b/daklib/dak_exceptions.py
@@ -44,13 +44,13 @@ dakerrors = {
     "UnknownFormatError":  """Exception raised for unknown Format: lines in changes files.""",
     "NoFilesFieldError":   """Exception raised for missing files field in dsc/changes.""",
     "CantOpenError":       """Exception raised when files can't be opened.""",
-    "CantOverwriteError":  """Exception raised when files cant be overwritten.""",
+    "CantOverwriteError":  """Exception raised when files can't be overwritten.""",
     "FileExistsError":     """Exception raised when destination file exists.""",
     "SendmailFailedError": """Exception raised when Sendmail invocation failed.""",
-    "NoFreeFilenameError": """Exception raised when no alternate filename was found."""
+    "NoFreeFilenameError": """Exception raised when no alternate filename was found.""",
+    "TransitionsError":    """Exception raised when transitions file can't be parsed."""
 }
 
-
 def construct_dak_exception(name, description):
     class Er(DakError):
         __doc__ = description