From 670d69f4019e3e6631fb8067b103e1191b7dd736 Mon Sep 17 00:00:00 2001 From: Colin Guthrie Date: Mon, 21 Apr 2014 19:44:34 +0000 Subject: mgagit: Don't report any error message when no change email address configured. When we will send to the i18n-reports ML info about i18n related changes we will omit the overall summary mail (it's too complext to produce a summary of only i18n changes) and thus we need to squash this error message which will not be nice for those pushing changes. --- deployment/mgagit/templates/git_multimail.py | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) (limited to 'deployment/mgagit') diff --git a/deployment/mgagit/templates/git_multimail.py b/deployment/mgagit/templates/git_multimail.py index cd8d30e6..ab192936 100755 --- a/deployment/mgagit/templates/git_multimail.py +++ b/deployment/mgagit/templates/git_multimail.py @@ -2253,13 +2253,7 @@ class Push(object): unhandled_sha1s = set(self.get_new_commits()) for change in self.changes: # Check if we've got anyone to send to - if not change.recipients: - sys.stderr.write( - '*** no recipients configured so no email will be sent\n' - '*** for %r update %s->%s\n' - % (change.refname, change.old.sha1, change.new.sha1,) - ) - else: + if change.recipients: sys.stderr.write('Sending notification emails to: %s\n' % (change.recipients,)) mailer.send(change.generate_email(self, body_filter), change.recipients) -- cgit v1.2.1