Re: [PATCH] Proof-of-concept patch to remember what the detached HEAD was

From: Jeff King <peff@peff.net>
Date: Oct 14, 2009, 23:09 UTC
Message-ID: <20091014230934.GC29664@coredump.intra.peff.net>
In-Reply-To: <7vws2xa9lu.fsf@alter.siamese.dyndns.org>

On Wed, Oct 14, 2009 at 03:34:05PM -0700, Junio C Hamano wrote:

Show 5 quoted lines

> > Agreed.  Presumably some expert mode config would imply -f 
> > automatically.
> 
> No, I do not want an expert mode.  I can probably live with "per session"
> setting, that makes me decide to set or not set it when I detach, though.

That makes the most sense to me. If "git checkout" could write metadata into HEAD (or into DETACH_HEAD, as in Daniel's patch), then checkout could record an "ok to commit" bit. And could also be used to change it after the fact. E.g.:

  $ git checkout --detach=commit origin/master
  $ git commit ;# should be ok

  $ git checkout --detach=examine origin/master
  $ git commit ;# complain
  $ git checkout --detach=commit HEAD
  $ git commit ;# ok

I guess something like "rebase" should detach with "ok to commit", since it is planning on attaching the commits later. I'm not sure about "git bisect". I guess probably it should be "not ok to commit" to be on the safe side, and then somebody can "git checkout --detach=commit" if they want to.

-Peff