Merge lp:~brian-murray/ubuntu-archive-tools/changer_email into lp:ubuntu-archive-tools

Proposed by Brian Murray
Status: Merged
Merged at revision: 1229
Proposed branch: lp:~brian-murray/ubuntu-archive-tools/changer_email
Merge into: lp:ubuntu-archive-tools
Diff against target: 12 lines (+2/-0)
1 file modified
phased-updater (+2/-0)
To merge this branch: bzr merge lp:~brian-murray/ubuntu-archive-tools/changer_email
Reviewer Review Type Date Requested Status
Ubuntu Package Archive Administrators Pending
Review via email: mp+370447@code.launchpad.net

Description of the change

This fixes this:

Traceback (most recent call last):
  File "/home/ubuntu-archive/ubuntu-archive-tools/phased-updater", line 728, in <module>
    main()
  File "/home/ubuntu-archive/ubuntu-archive-tools/phased-updater", line 695, in main
    create_email_notifications(releases, issues)
  File "/home/ubuntu-archive/ubuntu-archive-tools/phased-updater", line 279, in create_email_notifications
    if not changer_email:
UnboundLocalError: local variable 'changer_email' referenced before assignment

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'phased-updater'
2--- phased-updater 2018-09-18 14:07:35 +0000
3+++ phased-updater 2019-07-22 19:37:44 +0000
4@@ -261,6 +261,8 @@
5 signer_name = signer.name
6 # use the changes file as a backup method for determining email addresses
7 changes_file_url = spph.changesFileUrl()
8+ changer_name = ''
9+ changer_email = ''
10 try:
11 changes_file = urlopen(changes_file_url)
12 for line in changes_file.readlines():

Subscribers

People subscribed via source and target branches