From: Junio C Hamano Date: Tue, 24 May 2011 17:08:29 +0000 (-0700) Subject: checkout: make advice when reattaching the HEAD less loud X-Git-Tag: v1.7.7-rc0~125^2 X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=f807b3dcab9cafd76936d24936b8353f99b2ae6d;p=git.git checkout: make advice when reattaching the HEAD less loud When switching away from a detached HEAD with "git checkout", we give a listing of the commits about to be lost, and then tell how to resurrect them since 8e2dc6a (commit: give final warning when reattaching HEAD to leave commits behind, 2011-02-18). This is a good safety measure for people who are not comfortable with the detached HEAD state, but the advice on how to keep the state you just left was given even to those who set advice.detachedHead to false. Keep the warning and informational commit listing, but honor the setting of advice.detachedHead to squelch the advice. Helped-by: Jeff King Signed-off-by: Junio C Hamano --- diff --git a/builtin/checkout.c b/builtin/checkout.c index e44364c55..90870d953 100644 --- a/builtin/checkout.c +++ b/builtin/checkout.c @@ -634,14 +634,17 @@ static void suggest_reattach(struct commit *commit, struct rev_info *revs) "Warning: you are leaving %d commit%s behind, " "not connected to\n" "any of your branches:\n\n" - "%s\n" - "If you want to keep them by creating a new branch, " - "this may be a good time\nto do so with:\n\n" - " git branch new_branch_name %s\n\n", + "%s\n", lost, ((1 < lost) ? "s" : ""), - sb.buf, - sha1_to_hex(commit->object.sha1)); + sb.buf); strbuf_release(&sb); + + if (advice_detached_head) + fprintf(stderr, + "If you want to keep them by creating a new branch, " + "this may be a good time\nto do so with:\n\n" + " git branch new_branch_name %s\n\n", + sha1_to_hex(commit->object.sha1)); } /*