From: Joerg Jaspert <joerg@debian.org>
Date: Fri, 1 Jan 2010 10:38:08 +0000 (+0100)
Subject: queue, check_source
X-Git-Url: https://git.decadent.org.uk/gitweb/?a=commitdiff_plain;h=15356616ed745cb6893c59e371aeff08fac9f430;p=dak.git

queue, check_source

only bail out if we have a non-source upload.
do not bail out if self.pkg.orig_files is empty.
We either have a bug filling orig_files where we only set this when the files
already have been in the archive, or we intended to use orig_files to show
us files already in the archive. Currently it looks like the latter, and then
it is wrong to check on that in check_source, as that would mean we skip one
check (and also extraction of .debinfo information for the BTS!) for every
upload that comes with totally new files. (Imagine a native package).

Signed-off-by: Joerg Jaspert <joerg@debian.org>
---

diff --git a/daklib/queue.py b/daklib/queue.py
index 00238102..5b287172 100755
--- a/daklib/queue.py
+++ b/daklib/queue.py
@@ -1096,9 +1096,7 @@ class Upload(object):
     def check_source(self):
         # Bail out if:
         #    a) there's no source
-        # or c) the orig files are MIA
-        if not self.pkg.changes["architecture"].has_key("source") \
-           or len(self.pkg.orig_files) == 0:
+        if not self.pkg.changes["architecture"].has_key("source"):
             return
 
         tmpdir = utils.temp_dirname()