Whatever the difference between the summary and the subject field is: We

[dak.git] / daklib / queue_install.py

diff --git a/daklib/queue_install.py b/daklib/queue_install.py
old mode 100644 (file)
new mode 100755 (executable)
index 912fdcb..bc1b873
--- a/daklib/queue_install.py
+++ b/daklib/queue_install.py
@@ -69,12 +69,12 @@ def package_to_queue(u, summary, short_summary, queue, chg, session, announce=No
     u.check_override()
 
     # Send accept mail, announce to lists and close bugs
-    template = os.path.join(cnf["Dir::Templates"], announce)
-    u.update_subst()
-    u.Subst["__SUITE__"] = ""
-    mail_message = utils.TemplateSubst(u.Subst, template)
-    utils.send_mail(mail_message)
-    u.announce(short_summary, True)
+    if announce:
+        template = os.path.join(cnf["Dir::Templates"], announce)
+        u.update_subst()
+        mail_message = utils.TemplateSubst(u.Subst, template)
+        utils.send_mail(mail_message)
+        u.announce(short_summary, True)
 
 ################################################################################
 
@@ -165,7 +165,7 @@ def is_autobyhand(u):
 
 def do_autobyhand(u, summary, short_summary, chg, session):
     print "Attempting AUTOBYHAND."
-    byhandleft = True
+    byhandleft = False
     for f, entry in u.pkg.files.items():
         byhandfile = f
 
@@ -187,7 +187,7 @@ def do_autobyhand(u, summary, short_summary, chg, session):
 
         if result == 0:
             os.unlink(byhandfile)
-            del entry
+            del u.pkg.files[f]
         else:
             print "Error processing %s, left as byhand." % (f)
             byhandleft = True