Re: [PATCH] Detached HEAD (experimental)

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Junio C Hamano
Date: Tuesday, January 9, 2007 - 3:37 pm

"J. Bruce Fields" <bfields@fieldses.org> writes:


I do not want to think about the consequences of adding more
cruft under .git/ directory.  For example, should PREV be
noticed by fsck and prune?  What should various forms of
'git-reset' do with it?  How does it interact with 'git-bisect'?

Being able to test merge or even make commits without being on a
branch is vastly useful.  It might or might not lead to anywhere
even after you make a handful commits -- and I would imagine
that it would be very handy to be able to be lazy and not having
to decide if it is worth a new branch.

But that may be just my imagination; I generally prefer any
feature that allows me to defer decision over something that
makes me decide early.  If Carl wants to do a patch to teach
'git-commit' (and all other things that can create commits) not
to do things from working in a detached HEAD, I would probably
not opposed to it too much, but I am fairly certain that I won't
be coding it myself.

It's tempting to forget about this whole "safety" business.
Because we allow "reset --hard" and other forms of operations
that can lose history if they were done while on a branch, only
giving the safety to "git checkout" feels somewhat silly.  And
the primary motive for detached HEAD as I understand it is for
sightseeing, and not allowing "reset --hard" to jump around is
just plain silly.

That is, after:

	git checkout v1.4.0

you are not on any branch, and we would still allow

	git reset --hard v1.2.0

which is exactly the same as:

	git checkout v1.2.0

You can still say:

	git checkout master

and we do not even check.

Which makes the "merge-base --check-ancestry" stuff I did last
night pretty much unnecessary, but that's Ok.  It will find
other uses.

-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] Detached HEAD (experimental), Junio C Hamano, (Tue Jan 2, 12:45 am)
Re: [PATCH] Detached HEAD (experimental), Edgar Toernig, (Tue Jan 2, 12:59 pm)
Re: [PATCH] Detached HEAD (experimental), Carl Worth, (Tue Jan 2, 2:56 pm)
Re: [PATCH] Detached HEAD (experimental), Junio C Hamano, (Tue Jan 2, 3:44 pm)
[PATCH] git-branch: show detached HEAD, Lars Hjemli, (Tue Jan 2, 4:22 pm)
Re: [PATCH] Detached HEAD (experimental), Carl Worth, (Tue Jan 2, 4:34 pm)
Re: [PATCH] Detached HEAD (experimental), Junio C Hamano, (Tue Jan 2, 7:45 pm)
Re: [PATCH] git-branch: show detached HEAD, Shawn O. Pearce, (Tue Jan 2, 10:18 pm)
Re: [PATCH] git-branch: show detached HEAD, Junio C Hamano, (Wed Jan 3, 12:05 am)
Re: [PATCH] git-branch: show detached HEAD, Lars Hjemli, (Wed Jan 3, 12:37 am)
Re: [PATCH] Detached HEAD (experimental), Jeff King, (Wed Jan 3, 3:46 am)
Re: [PATCH] Detached HEAD (experimental), Jeff King, (Wed Jan 3, 4:59 am)
Re: [PATCH] Detached HEAD (experimental), Junio C Hamano, (Mon Jan 8, 4:19 am)
Re: [PATCH] Detached HEAD (experimental), Jeff King, (Mon Jan 8, 6:17 am)
Re: [PATCH] Detached HEAD (experimental), Junio C Hamano, (Mon Jan 8, 5:19 pm)
Re: [PATCH] Detached HEAD (experimental), Carl Worth, (Mon Jan 8, 5:43 pm)
Re: [PATCH] Detached HEAD (experimental), Junio C Hamano, (Mon Jan 8, 6:05 pm)
Re: [PATCH] Detached HEAD (experimental), Carl Worth, (Mon Jan 8, 6:15 pm)
Re: [PATCH] Detached HEAD (experimental), Shawn O. Pearce, (Mon Jan 8, 8:26 pm)
Re: [PATCH] Detached HEAD (experimental), Junio C Hamano, (Tue Jan 9, 12:07 am)
Re: [PATCH] Detached HEAD (experimental), Luben Tuikov, (Tue Jan 9, 1:12 am)
Re: [PATCH] Detached HEAD (experimental), Jeff King, (Tue Jan 9, 7:21 am)
Re: [PATCH] Detached HEAD (experimental), Junio C Hamano, (Tue Jan 9, 2:20 pm)
Re: [PATCH] Detached HEAD (experimental), J. Bruce Fields, (Tue Jan 9, 2:31 pm)
Re: [PATCH] Detached HEAD (experimental), Carl Worth, (Tue Jan 9, 2:43 pm)
Re: [PATCH] Detached HEAD (experimental), J. Bruce Fields, (Tue Jan 9, 2:53 pm)
Re: [PATCH] Detached HEAD (experimental), Junio C Hamano, (Tue Jan 9, 3:37 pm)
Re: [PATCH] Detached HEAD (experimental), Shawn O. Pearce, (Tue Jan 9, 4:39 pm)
Re: [PATCH] Detached HEAD (experimental), Shawn O. Pearce, (Tue Jan 9, 4:44 pm)
Re: [PATCH] Detached HEAD (experimental), Linus Torvalds, (Tue Jan 9, 4:46 pm)
Re: [PATCH] Detached HEAD (experimental), Junio C Hamano, (Tue Jan 9, 5:10 pm)
Re: [PATCH] Detached HEAD (experimental), Shawn O. Pearce, (Tue Jan 9, 5:18 pm)
Re: [PATCH] Detached HEAD (experimental), Carl Worth, (Tue Jan 9, 5:51 pm)
Re: [PATCH] Detached HEAD (experimental), Linus Torvalds, (Tue Jan 9, 5:54 pm)
Re: [PATCH] Detached HEAD (experimental), Junio C Hamano, (Wed Jan 10, 1:02 am)
Re: [PATCH] Detached HEAD (experimental), Andy Parkins, (Wed Jan 10, 2:04 am)
Re: [PATCH] Detached HEAD (experimental), Shawn O. Pearce, (Wed Jan 10, 2:05 am)
Re: [PATCH] Detached HEAD (experimental), Andreas Ericsson, (Wed Jan 10, 2:08 am)
Re: [PATCH] Detached HEAD (experimental), Junio C Hamano, (Wed Jan 10, 2:33 am)
Re: [PATCH] Detached HEAD (experimental), Junio C Hamano, (Wed Jan 10, 2:40 am)
Re: [PATCH] Detached HEAD (experimental), Junio C Hamano, (Wed Jan 10, 2:46 am)
Re: [PATCH] Detached HEAD (experimental), Andy Parkins, (Wed Jan 10, 3:10 am)
Re: [PATCH] Detached HEAD (experimental), Shawn O. Pearce, (Wed Jan 10, 3:25 am)
Re: [PATCH] Detached HEAD (experimental), Jeff King, (Wed Jan 10, 7:04 am)
Re: [PATCH] Detached HEAD (experimental), Junio C Hamano, (Wed Jan 10, 9:18 am)
Re: [PATCH] Detached HEAD (experimental), Daniel Barkalow, (Wed Jan 10, 9:30 am)
Re: [PATCH] Detached HEAD (experimental), Junio C Hamano, (Wed Jan 10, 5:34 pm)
Re: [PATCH] Detached HEAD (experimental), J. Bruce Fields, (Wed Jan 10, 9:31 pm)
Re: [PATCH] Detached HEAD (experimental), Andreas Ericsson, (Thu Jan 11, 2:45 am)