Merge lp:~jelmer/brz/branches-url into lp:brz

Proposed by Jelmer Vernooij
Status: Merged
Approved by: Jelmer Vernooij
Approved revision: no longer in the source branch.
Merge reported by: The Breezy Bot
Merged at revision: not available
Proposed branch: lp:~jelmer/brz/branches-url
Merge into: lp:brz
Diff against target: 12 lines (+1/-1)
1 file modified
breezy/builtins.py (+1/-1)
To merge this branch: bzr merge lp:~jelmer/brz/branches-url
Reviewer Review Type Date Requested Status
Martin Packman Approve
Review via email: mp+340765@code.launchpad.net

Commit message

Compare user_url branch attribute in cmd_branches, when finding active branch.

Description of the change

Compare user_url branch attribute in cmd_branches, when finding active branch.

To post a comment you must log in.
Revision history for this message
Martin Packman (gz) wrote :

Did this cause a test to fail, or is it just a correctness change?

review: Approve
Revision history for this message
Jelmer Vernooij (jelmer) wrote :

> Did this cause a test to fail, or is it just a correctness change?
This is a correctness fix. It does change the behaviour for "bzr branches", but just for git branches. I'm adding a separate test in brz-git.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'breezy/builtins.py'
--- breezy/builtins.py 2018-03-02 00:38:34 +0000
+++ breezy/builtins.py 2018-03-05 21:12:44 +0000
@@ -1652,7 +1652,7 @@
1652 if name == "":1652 if name == "":
1653 continue1653 continue
1654 active = (active_branch is not None and1654 active = (active_branch is not None and
1655 active_branch.base == branch.base)1655 active_branch.user_url == branch.user_url)
1656 names[name] = active1656 names[name] = active
1657 # Only mention the current branch explicitly if it's not1657 # Only mention the current branch explicitly if it's not
1658 # one of the colocated branches1658 # one of the colocated branches

Subscribers

People subscribed via source and target branches