Skip to content

Commit

Permalink
Merge branch 'jm/branch-rename-nothing-error'
Browse files Browse the repository at this point in the history
"git branch -m" without any argument noticed an error, but with an
incorrect error message.

* jm/branch-rename-nothing-error:
  branch: give better message when no names specified for rename
  • Loading branch information
Junio C Hamano committed Apr 3, 2013
2 parents ed23f31 + d1520c4 commit 8054b9a
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 3 additions & 1 deletion builtin/branch.c
Original file line number Diff line number Diff line change
Expand Up @@ -904,7 +904,9 @@ int cmd_branch(int argc, const char **argv, const char *prefix)
if (edit_branch_description(branch_name))
return 1;
} else if (rename) {
if (argc == 1)
if (!argc)
die(_("branch name required"));
else if (argc == 1)
rename_branch(head, argv[0], rename > 1);
else if (argc == 2)
rename_branch(argv[0], argv[1], rename > 1);
Expand Down
2 changes: 1 addition & 1 deletion t/t3200-branch.sh
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ test_expect_success 'git branch l should work after branch l/m has been deleted'

test_expect_success 'git branch -m dumps usage' '
test_expect_code 128 git branch -m 2>err &&
test_i18ngrep "too many branches for a rename operation" err
test_i18ngrep "branch name required" err
'

test_expect_success 'git branch -m m m/m should work' '
Expand Down

0 comments on commit 8054b9a

Please sign in to comment.