Merge lp:~jelmer/brz/no-colo-name 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/no-colo-name |
Merge into: | lp:brz |
Diff against target: |
48 lines (+6/-6) 3 files modified
breezy/builtins.py (+1/-3) breezy/tests/blackbox/test_branch.py (+4/-3) breezy/urlutils.py (+1/-0) |
To merge this branch: | bzr merge lp:~jelmer/brz/no-colo-name |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Martin Packman | Approve | ||
Review via email: mp+354946@code.launchpad.net |
Commit message
Don't use colocated branch names for derived branch directory names.
Description of the change
When running "bzr branch http://
This is a bad idea for a couple of reasons:
* unpredictable
* most of the time, the remote's name is going to be "master" anyway
Instead, just derive the local name from the last part of the remote URL, as we do when there are no colocated branches involved.
To post a comment you must log in.
Makes sense.