rmbranch should refuse removing active branch
Bug #922953 reported by
Jelmer Vernooij
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Bazaar |
Fix Released
|
Medium
|
Jelmer Vernooij | ||
| 2.5 |
Fix Released
|
Medium
|
Unassigned | ||
| bzr (Ubuntu) |
Fix Released
|
Medium
|
Jelmer Vernooij | ||
| Precise |
Fix Released
|
Medium
|
Jelmer Vernooij | ||
Bug Description
"bzr rmbranch" shouldn't let the user shoot themselves in the foot by removing the currently active branch.
Instead, it should print an error message if the user tries to remove the active branch and only actually remove the branch when --force is specified.
Related branches
lp:~jelmer/bzr/rmbranch-active
- Martin Packman (community): Approve
-
Diff: 130 lines (+46/-10)3 files modifiedbzrlib/builtins.py (+12/-2)
bzrlib/tests/blackbox/test_rmbranch.py (+31/-8)
doc/en/release-notes/bzr-2.6.txt (+3/-0)
- bzr-core: Pending requested
-
Diff: 129 lines (+46/-10)3 files modifiedbzrlib/builtins.py (+12/-2)
bzrlib/tests/blackbox/test_rmbranch.py (+31/-8)
doc/en/release-notes/bzr-2.5.txt (+3/-0)
lp:~gz/bzr/2.5_backport_rmbranch_fixes
- Jelmer Vernooij (community): Approve (code)
-
Diff: 220 lines (+95/-23)4 files modifiedbzrlib/builtins.py (+47/-14)
bzrlib/tests/blackbox/test_rmbranch.py (+41/-8)
bzrlib/tests/blackbox/test_switch.py (+1/-1)
doc/en/release-notes/bzr-2.5.txt (+6/-0)
| Changed in bzr: | |
| status: | Confirmed → In Progress |
| assignee: | nobody → Jelmer Vernooij (jelmer) |
| Changed in bzr: | |
| milestone: | none → 2.5.0 |
| Changed in bzr: | |
| milestone: | 2.5.0 → none |
| Changed in bzr: | |
| milestone: | none → 2.6b1 |
| status: | In Progress → Fix Released |
| Changed in bzr (Ubuntu): | |
| status: | New → Incomplete |
| status: | Incomplete → Fix Released |
| importance: | Undecided → Medium |
| assignee: | nobody → Jelmer Vernooij (jelmer) |
| Changed in bzr (Ubuntu Precise): | |
| status: | New → In Progress |
| importance: | Undecided → Medium |
| assignee: | nobody → Jelmer Vernooij (jelmer) |
Revision history for this message
| Chris Halse Rogers (raof) wrote Please test proposed package | #1 |
| Changed in bzr (Ubuntu Precise): | |
| status: | In Progress → Fix Committed |
| tags: | added: verification-needed |
Revision history for this message
| Andrew Starr-Bochicchio (andrewsomething) wrote | #2 |
| tags: |
added: verification-done removed: verification-needed |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #3 |
| Changed in bzr (Ubuntu Precise): | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Hello Jelmer, or anyone else affected,
Accepted bzr into precise-proposed. The package will build now and be available in a few hours. Please test and give feedback here. See https:/