]> Frank Brehm's Git Trees - profitbricks/jenkins-build-scripts.git/commitdiff
cleanup
authorHolger Levsen <holger@layer-acht.org>
Sat, 17 Sep 2011 18:16:44 +0000 (20:16 +0200)
committerHolger Levsen <holger@layer-acht.org>
Sat, 17 Sep 2011 18:16:44 +0000 (20:16 +0200)
debian_build.py

index df31a180cee624a3bacbbaa370adac29c929f964..80f3b794c63bb5a4af7fbf7c5fa05ce192ce6ecb 100755 (executable)
@@ -339,6 +339,11 @@ if __name__ == '__main__':
         else:
             logger.info('git log was successfull')
 
+        new_log = git_log.stdout.read()
+        if new_log == '': 
+            logger.info('git log since yesterday is empty, aborting build.')
+            sys.exit(0)
+
         cmd = [
                 '/usr/bin/dch',
                 '--create',
@@ -349,13 +354,11 @@ if __name__ == '__main__':
                 '--distribution',
                 '%s' %(options.distribution),
                 '--',
-                '%s' %(''.join(git_log.stdout.read()))
+                '%s' %(''.join(new_log))
         ]
 
         logger.debug('Trying to call: %s' %(' '.join(cmd)))
 
-        new_log = git_log.stdout.read()
-
         dch = subprocess.Popen(
                 cmd,
                 shell=False,
@@ -373,13 +376,6 @@ if __name__ == '__main__':
             )
         logger.info('debian/changelog written')
 
-        if new_log == '': 
-            logger.info('git log since yesterday is empty, aborting build.')
-            sys.exit(0)
-        else:
-            logger.info('git log since yesterday: %s ' % new_log)
-
-
         # we need to commit here else git-buildpackage will use the existing debian/changelog...
         # TODO: Later we should investigate why the "--ignore-new" trick
         # did not work!