From 8aa7b4e1db41f1bf0ff27da01d51bb726d491c09 Mon Sep 17 00:00:00 2001 From: clime Date: Fri, 29 Jul 2016 11:26:02 +0200 Subject: [PATCH] Send Reviews Weekly from nobody@fedoraproject.org Added --email-from option for bz-review-report script and used this option in cron with "nobody@fedoraproject.org". --- roles/bz_review_report/files/bz-review-report.cron | 2 +- roles/bz_review_report/files/bz-review-report.py | 8 +++++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/roles/bz_review_report/files/bz-review-report.cron b/roles/bz_review_report/files/bz-review-report.cron index 0713039ba6..48fbc340f3 100644 --- a/roles/bz_review_report/files/bz-review-report.cron +++ b/roles/bz_review_report/files/bz-review-report.cron @@ -1 +1 @@ -8 10 * * 1 root /usr/bin/python /usr/local/bin/bz-review-report.py -r rev-com -u --email=devel@lists.fedoraproject.org &> /dev/null +8 10 * * 1 root /usr/bin/python /usr/local/bin/bz-review-report.py -r rev-com -u --email=devel@lists.fedoraproject.org --email-from=nobody@fedoraproject.org &> /dev/null diff --git a/roles/bz_review_report/files/bz-review-report.py b/roles/bz_review_report/files/bz-review-report.py index bc3ea72ac6..5b0032553f 100755 --- a/roles/bz_review_report/files/bz-review-report.py +++ b/roles/bz_review_report/files/bz-review-report.py @@ -208,6 +208,9 @@ if __name__ == '__main__': parser.add_option("-m", "--email", action="store", dest="email_address", help="If specified, send results to the given email address instead of generating 'output.txt' file.") + parser.add_option("-f", "--email-from", action="store", dest="email_from", + help="Optional. If specified, then this address will be used for From: field") + parser.add_option("-n", "--username", action="store", dest="username", help="BZ username") parser.add_option("-p", "--password", action="store", dest="password", help="BZ password") @@ -422,7 +425,10 @@ if __name__ == '__main__': file.close() subject = "Reviews Weekly" - me = socket.gethostname() + if options.email_from: + me = options.email_from + else: + me = socket.gethostname() you = options.email_address msg['Subject'] = subject