Re: [PATCH] Detached HEAD (experimental)

From: Junio C Hamano <junkio@cox.net>
Date: Jan 3, 2007, 02:45 UTC
Message-ID: <7vd55wsu8w.fsf@assigned-by-dhcp.cox.net>
In-Reply-To: <87odphgfzz.wl%cworth@cworth.org>

Carl Worth <cworth@cworth.org> writes:

> I don't actually prefer "no commit allowed". I just didn't want the
> user to have to explicitly disable the safety before being able to
> perform a checkout based on a tag.

Ok, then I think we are in agreement that the safety should be at the point where the user might leave the detached state.

Show 5 quoted lines

> I am still interested in this feature, so I will try to find time to
> come back with a revised version of your patch with the missing safety
> check (and without requiring -d on checkout). Thanks again for this
> initial take on the problem. (Though if anyone else beats me to it, I
> certainly will not be offended.)

Sounds good.

I do not mind losing -d, but I would suggest that there should be a message that says "you are no longer on any branch" after a checkout that makes the head detached. The user should be warned about such an unusal situation.