"bzr branches" should list colocated branches if supported
Bug #826820 reported by
Jelmer Vernooij
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Bazaar |
Fix Released
|
Medium
|
Jelmer Vernooij | ||
BzrTools |
Invalid
|
Medium
|
Jelmer Vernooij |
Bug Description
The fact that the current implementation in bzrtools scans the file system seems to be confusing a lot of users, especially because it takes a fair amount of time to run.
Now that there is support for colocated branches in the core, it would also be nice to be able to list all colocated branches without requiring a plugin. I think "bzr branches" should by default just call list_branches() on the current control directory, rather than recursively searching the file system.
Perhaps the existing "bzr branches" can be renamed to e.g. "bzr discover-branches"?
Related branches
lp://qastaging/~jelmer/bzr/cmd-branches
- Jonathan Riddell (community): Approve
-
Diff: 110 lines (+66/-2)4 files modifiedbzrlib/builtins.py (+17/-0)
bzrlib/tests/blackbox/__init__.py (+1/-0)
bzrlib/tests/blackbox/test_branches.py (+43/-0)
doc/en/release-notes/bzr-2.5.txt (+5/-2)
lp://qastaging/~jelmer/bzrtools/no-more-branches
Rejected
for merging
into
lp://qastaging/bzrtools
- Jonathan Riddell (community): Approve
- Aaron Bentley: Pending requested
-
Diff: 313 lines (+17/-91)13 files modified__init__.py (+0/-1)
branches.py (+0/-12)
bzrtools.py (+6/-33)
cbranch.py (+0/-1)
command_classes.py (+0/-18)
heads.py (+1/-2)
hunk_selector.py (+1/-1)
patch.py (+1/-2)
progress.py (+0/-2)
rspush.py (+0/-2)
shelf.py (+2/-3)
tests/blackbox.py (+0/-13)
tests/test_link_tree.py (+6/-1)
Changed in bzr: | |
status: | Triaged → Fix Released |
milestone: | none → 2.5b1 |
assignee: | nobody → Jelmer Vernooij (jelmer) |
Changed in bzrtools: | |
status: | New → In Progress |
assignee: | nobody → Jelmer Vernooij (jelmer) |
Changed in bzrtools: | |
status: | In Progress → Fix Committed |
importance: | Undecided → Medium |
Changed in bzrtools: | |
status: | Fix Committed → Invalid |
To post a comment you must log in.
+1