On Wed, Jan 31, 2007 at 01:59:37PM -0500, Nicolas Pitre wrote: > > I would also be happy with warning (and > > probably blocking without -f) moving from 3 to 1, which is the actual > > dangerous thing. > And that is already what is happening. Doh! I'm a complete moron. Sorry, but I thought we were _not_ warning there in favor of the warning at time of detachment. I even did a test, but I botched it. So please, accept my apology and assume I have hit myself over the head with the clue stick several times. Warning at commit time _is_ stupid, since we can complain at the correct time. > Let's fix the warning then. But it must stay just because it is > important that the user know _why_ and _when_ the head became detached. > Realizing that head is detached later is far more confusing if the user > just don't know how that happened. OK, I completely see your point now; it doesn't have to be a _warning_ per se, but rather to let the user know this is when the state changed (so that later if they do get a warning, it makes more sense). > > > But making a warning at commit time is wrong. It is completely > > > disconnected from the actual issue and I think it'd create more > > > confusion because there is in fact nothing to worry about at the moment > > > the commit is made. The very fact that you think yourself that a > > > warning should be displayed at commit time indicates to me that you > > > might be a bit confused yourself and such warning if present at commit > > > time wouldn't help clearing that confusion at all. > > > > I think you are proving my point here. If you think warning at commit > > time is too early, then how is warning _before_ that (when we detach) > > not too early? > > Did I say anything about it being too early? > > I say that it is unnecessary and redundent, and that it would create > more confusion than it clears. You said "...there is in fact nothing to worry about at the moment the commit is made." My point is that there is in fact nothing to worry about at the moment that you detach, thus why should one get a warning and not the other. But I agree that if you want the later warning to make sense, it might be helpful to note that point (and I think it's getting too fancy to tuck away that information and have the actual warning say "When you moved your HEAD to foo~32, you were no longer on a branch, therefore...") So IOW, I think I agree with you now. :) Again, sorry for the (my) confusion. -Peff - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html