Merge lp:~jelmer/bzrtools/fix-nick-reference into lp:bzrtools

Proposed by Jelmer Vernooij
Status: Merged
Merged at revision: 779
Proposed branch: lp:~jelmer/bzrtools/fix-nick-reference
Merge into: lp:bzrtools
Diff against target: 16 lines (+2/-1)
1 file modified (+2/-1)
To merge this branch: bzr merge lp:~jelmer/bzrtools/fix-nick-reference
Reviewer Review Type Date Requested Status
Aaron Bentley Approve
Review via email:

Description of the change

Patch from Andi Albrecht in #263065

This fixes an UnboundLocalError if a revision id contains a committer but is not present in the repository.

To post a comment you must log in.
Revision history for this message
Aaron Bentley (abentley) wrote :

I prefer branches to have test cases. I've added one and merged. Thanks.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file ''
2--- 2011-03-13 16:06:41 +0000
3+++ 2011-06-28 15:03:30 +0000
4@@ -153,10 +153,11 @@
5 return new_ancestors
7 def get_rev_info(rev_id, source):
8- """Return the committer, message, and date of a revision."""
9+ """Return the committer, message, nick and date of a revision."""
10 committer = None
11 message = None
12 date = None
13+ nick = None
14 if rev_id == 'null:':
15 return None, 'Null Revision', None, None
16 try:


People subscribed via source and target branches