chiark / gitweb /
Don't special-case the "master" branch during branch delete
authorKarl Hasselström <kha@treskal.com>
Sat, 22 Sep 2007 07:54:25 +0000 (09:54 +0200)
committerKarl Hasselström <kha@treskal.com>
Sat, 22 Sep 2007 07:54:25 +0000 (09:54 +0200)
It's unintuitive, unsymmetric and doesn't work if there is no
"master". Instead, prohibit deletion of the current branch.

Signed-off-by: Karl Hasselström <kha@treskal.com>
stgit/commands/branch.py

index 2d491d50a86c837051249a0f17772577d934ed56..8c999363cb99ef586b441cfd30976ee902c7a7b2 100644 (file)
@@ -96,17 +96,10 @@ def __delete_branch(doomed_name, force = False):
     out.start('Deleting branch "%s"' % doomed_name)
 
     if __is_current_branch(doomed_name):
-        check_local_changes()
-        check_conflicts()
-        check_head_top_equal()
-
-        if doomed_name != 'master':
-            git.switch_branch('master')
+        raise CmdException('Cannot delete the current branch')
 
     doomed.delete(force)
-
-    if doomed_name != 'master':
-        git.delete_branch(doomed_name)
+    git.delete_branch(doomed_name)
 
     out.done()