From: Joerg Jaspert <joerg@debian.org>
Date: Sun, 12 Aug 2012 13:32:53 +0000 (+0200)
Subject: Merge remote-tracking branch 'ansgar/pu/multiarchive-2'
X-Git-Url: https://git.decadent.org.uk/gitweb/?p=dak.git;a=commitdiff_plain;h=61d1e2b14d61c045dcc07cd177dec49af0c41c45;hp=039b87c9a0d68c3acd4cd893396334d6f3ea06a0

Merge remote-tracking branch 'ansgar/pu/multiarchive-2'

* ansgar/pu/multiarchive-2:
  dak/process_upload.py: only send mail to tracking server if accepted to real suite
  templates/process-unchecked.accepted: use only defined subst variables

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

diff --git a/dak/process_upload.py b/dak/process_upload.py
index 74229e2f..bea8a10e 100755
--- a/dak/process_upload.py
+++ b/dak/process_upload.py
@@ -302,24 +302,23 @@ def accept(directory, upload):
     utils.send_mail(message)
 
     # send mail to announce lists and tracking server
-    subst = subst_for_upload(upload)
-    announce = set()
-    for suite in upload.final_suites:
-        if suite.policy_queue is not None:
-            continue
-        announce.update(suite.announce or [])
-    announce_address = ", ".join(announce)
-
-    tracking = cnf.get('Dinstall::TrackingServer')
-    if tracking and 'source' in upload.changes.architectures:
-        announce_address = '{0}\nBcc: {1}@{2}'.format(announce_address, control['Source'], tracking)
+    if  accepted_to_real_suite:
+        subst = subst_for_upload(upload)
+        announce = set()
+        for suite in upload.final_suites:
+            if suite.policy_queue is not None:
+                continue
+            announce.update(suite.announce or [])
+        announce_address = ", ".join(announce)
 
-    subst['__ANNOUNCE_LIST_ADDRESS__'] = announce_address
+        tracking = cnf.get('Dinstall::TrackingServer')
+        if tracking and 'source' in upload.changes.architectures:
+            announce_address = '{0}\nBcc: {1}@{2}'.format(announce_address, control['Source'], tracking)
 
-    message = utils.TemplateSubst(subst, os.path.join(cnf['Dir::Templates'], 'process-unchecked.announce'))
-    utils.send_mail(message)
+        subst['__ANNOUNCE_LIST_ADDRESS__'] = announce_address
 
-    del subst['__ANNOUNCE_LIST_ADDRESS__']
+        message = utils.TemplateSubst(subst, os.path.join(cnf['Dir::Templates'], 'process-unchecked.announce'))
+        utils.send_mail(message)
 
     # Only close bugs for uploads that were not redirected to a policy queue.
     # process-policy will close bugs for those once they are accepted.
diff --git a/templates/process-unchecked.accepted b/templates/process-unchecked.accepted
index 09b5e0c9..0fc0318b 100644
--- a/templates/process-unchecked.accepted
+++ b/templates/process-unchecked.accepted
@@ -9,8 +9,7 @@ Content-Type: text/plain; charset="utf-8"
 Content-Transfer-Encoding: 8bit
 Subject: __CHANGES_FILENAME__ ACCEPTED into __SUITE__
 
-__REJECT_MESSAGE__
 Accepted:
-__SUMMARY__
+__FILE_CONTENTS__
 
 Thank you for your contribution to __DISTRO__.