From: Junio C Hamano <gitster@pobox.com>
Date: Sat, 16 May 2009 19:57:44 GMT
Subject: Re: [PATCH] builtin-checkout: Don't tell user that HEAD has moved before it has
Message-ID: <7vr5yolt1j.fsf@alter.siamese.dyndns.org>
In-Reply-To: <20090516175444.GA1000@cumin.applehq.eu>

Daniel Cordero <theappleman@gmail.com> writes:

> Previously, checkout would tell the user this message before moving HEAD,
> without regard to whether the upcoming move will result in success.
> If the move failed, this causes confusion.
>
> Show the message after the move, unless the move failed.
>
> Signed-off-by: Daniel Cordero <theappleman@gmail.com>

Thanks; the patch looks obviously good.  Can we have tests?