From: Brandon Casey Date: Sun, 22 Mar 2009 00:09:17 +0000 (-0500) Subject: git-branch: display "was sha1" on branch deletion rather than just "sha1" X-Git-Tag: v1.6.2.2~4^2 X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=76aac715465b7678ca5b834ccbdcb88a3ab8a9f4;p=git.git git-branch: display "was sha1" on branch deletion rather than just "sha1" Make it more pleasant to read about a branch deletion by adding "was". Jeff King suggested this, and I ignored it. He was right. Update t3200 test again to match the change in output. Signed-off-by: Brandon Casey Signed-off-by: Junio C Hamano --- diff --git a/builtin-branch.c b/builtin-branch.c index 02fa38fd3..23b6949fe 100644 --- a/builtin-branch.c +++ b/builtin-branch.c @@ -165,7 +165,7 @@ static int delete_branches(int argc, const char **argv, int force, int kinds) ret = 1; } else { struct strbuf buf = STRBUF_INIT; - printf("Deleted %sbranch %s (%s).\n", remote, argv[i], + printf("Deleted %sbranch %s (was %s).\n", remote, argv[i], find_unique_abbrev(sha1, DEFAULT_ABBREV)); strbuf_addf(&buf, "branch.%s", argv[i]); if (git_config_rename_section(buf.buf, NULL) < 0) diff --git a/t/t3200-branch.sh b/t/t3200-branch.sh index 61a2010f5..1b1e9ece5 100755 --- a/t/t3200-branch.sh +++ b/t/t3200-branch.sh @@ -195,7 +195,7 @@ test_expect_success 'test deleting branch deletes branch config' \ test_expect_success 'test deleting branch without config' \ 'git branch my7 s && sha1=$(git rev-parse my7 | cut -c 1-7) && - test "$(git branch -d my7 2>&1)" = "Deleted branch my7 ($sha1)."' + test "$(git branch -d my7 2>&1)" = "Deleted branch my7 (was $sha1)."' test_expect_success 'test --track without .fetch entries' \ 'git branch --track my8 &&