honour No-Action in clean suites

Signed-off-by: NMark Hymers <mhy@debian.org>

dak/clean_suites.py

@@ -68,8 +68,11 @@ SELECT b.file, f.filename FROM binaries b, files f
 
     for i in q.fetchall():
         Logger.log(["set lastused", i[1]])
+        if not Options["No-Action"]:
             session.execute("UPDATE files SET last_used = :lastused WHERE id = :fileid AND last_used IS NULL",
                             {'lastused': now_date, 'fileid': i[0]})
+
+    if not Options["No-Action"]:
         session.commit()
 
     # Check for any binaries which are marked for eventual deletion
@@ -82,7 +85,10 @@ SELECT b.file, f.filename FROM binaries b, files f
 
     for i in q.fetchall():
         Logger.log(["unset lastused", i[1]])
+        if not Options["No-Action"]:
             session.execute("UPDATE files SET last_used = NULL WHERE id = :fileid", {'fileid': i[0]})
+
+    if not Options["No-Action"]:
         session.commit()
 
 ########################################
@@ -109,6 +115,7 @@ SELECT s.id, s.file, f.filename FROM source s, files f
 
         # Mark the .dsc file for deletion
         Logger.log(["set lastused", dsc_fname])
+        if not Options["No-Action"]:
             session.execute("""UPDATE files SET last_used = :last_used
                                 WHERE id = :dscfileid AND last_used IS NULL""",
                             {'last_used': now_date, 'dscfileid': dsc_file_id})
@@ -123,10 +130,12 @@ SELECT s.id, s.file, f.filename FROM source s, files f
             y = session.execute("SELECT id FROM dsc_files d WHERE d.file = :fileid", {'fileid': file_id})
             if len(y.fetchall()) == 1:
                 Logger.log(["set lastused", file_name])
+                if not Options["No-Action"]:
                     session.execute("""UPDATE files SET last_used = :lastused
                                        WHERE id = :fileid AND last_used IS NULL""",
                                     {'lastused': now_date, 'fileid': file_id})
 
+    if not Options["No-Action"]:
         session.commit()
 
     # Check for any sources which are marked for deletion but which
@@ -143,9 +152,11 @@ SELECT f.id, f.filename FROM source s, files f, dsc_files df
 
     for i in q.fetchall():
         Logger.log(["unset lastused", i[1]])
+        if not Options["No-Action"]:
             session.execute("UPDATE files SET last_used = NULL WHERE id = :fileid",
                             {'fileid': i[0]})
 
+    if not Options["No-Action"]:
         session.commit()
 
 ########################################
@@ -174,9 +185,11 @@ SELECT id, filename FROM files f
         for x in ql:
             utils.warn("orphaned file: %s" % x)
             Logger.log(["set lastused", x[1], "ORPHANED FILE"])
+            if not Options["No-Action"]:
                  session.execute("UPDATE files SET last_used = :lastused WHERE id = :fileid",
                                  {'lastused': now_date, 'fileid': x[0]})
 
+        if not Options["No-Action"]:
             session.commit()
 
 def clean_binaries(now_date, delete_date, max_delete, session):