Re: [RFC] Detached-HEAD reminder on commit?
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Sep 2, 2008, 21:58 UTC
- Message-ID
- <7vr682i50g.fsf@gitster.siamese.dyndns.org>
- In-Reply-To
- <200809022339.20123.johan@herland.net>
Johan Herland <johan@herland.net> writes:
Show 23 quoted lines
> On Tuesday 02 September 2008, Stephan Beyer wrote: >> Junio C Hamano wrote: >> > Pieter de Bie <pdebie@ai.rug.nl> writes: >> > > Vienna:git pieter$ ./git commit --allow-empty -m"test" >> > > Created commit 6ce62c8b: test >> > > You are on a detached head, so this commit has not been recorded in a >> > > branch. If you don't want to lose this commit, checkout a branch and >> > > then run: git merge 6ce62c8bfcfb341106f3587d1c141c3955c2544c >> > > >> > > Are there any comments to this / strong opinions against such a >> > > change? >> > >> > Unconditionally doing this is too loud for my taste. You probably can >> > do this in your post-commit hook. >> >> Well, Pieter probably can do this in his post-commit hook. But I think >> this is useful for usability... especially for beginners who might not >> even know what a hook is. ;) > > I'm not sure I like this personally, but if we _really_ don't want newbies > to shoot themselves in the foot, we could make "git commit" fail on a > detached HEAD unless the user has indicated that s/he knows what's going > on; i.e. something like this:
If we _really_ don't want newbies to shoot themselves in the foot, we probably can issue a loud warning when they detach there HEAD.
Oh, wait,... we already do that.