]> Frank Brehm's Git Trees - profitbricks/jenkins-build-scripts.git/commitdiff
update debian_build: fix the second (same) conversion error
authorMathias Klette <mathias.klette@profitbricks.com>
Wed, 24 Apr 2013 18:33:07 +0000 (20:33 +0200)
committerMathias Klette <mathias.klette@profitbricks.com>
Wed, 24 Apr 2013 18:33:07 +0000 (20:33 +0200)
- define the string only once
- use this variable for both invokations then

debian_build.py

index 94d54fda0ef582df965b719ba348ca3c2afe61d6..94689d50873e4ae291cce378d4bf8226e87c3bf7 100755 (executable)
@@ -418,15 +418,12 @@ if __name__ == '__main__':
     # enforce correct distribution in debian/changelog for master and hotfix branches
     allowed_production_distros=('stable', 'stable-proposed-updates', 'production', 'production-proposed-updates')
     if (GIT_BRANCH_NAME == 'master' or GIT_BRANCH_NAME.startswith('hotfix/')) and not distribution in allowed_production_distros:
-        logger.error(
-            'Distribution %s in debian/changelog not listed in %s' %(
+        message = 'Distribution %s in debian/changelog not listed in %s' %(
             distribution,
             allowed_production_distros
-            ))
-        raise Exception(
-            'Distribution %s in debian/changelog not listed in %s' %(
-            allowed_production_distros,
-            ))
+            )
+        logger.error(message)
+        raise Exception(message)
         exit_error()
 
     DPUT_OPTIONS = DPUT_OPTIONS_DEFAULT