Re: [PATCH] tone down the detached head warning

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Jeff King <peff@...>
Cc: Nicolas Pitre <nico@...>, Junio C Hamano <junkio@...>, <git@...>
Date: Wednesday, January 31, 2007 - 7:54 pm

On Wed, 31 Jan 2007 18:19:43 -0500, Jeff King wrote:
...

Getting rid of the word "Warning" and naming the remote-tracking
branch instead of saying "not on ANY branch", (my, why should git ever
yell like that?), are definitely improvements.

But they're fairly incremental. The fact is that the user is doing a
very simple operation here, (just checking out a state for which git
already has a name), and the user is given 3 lines of text to read and
try to understand, (what the heck is a "remote tracking branch"
anyway?). It still looks to me like the kind of thing that promotes a
"git is hard to use" conception.

But, back to the original use case that brought this up. I did botch
something in the original description. The "git clone; git checkout
origin/branch-name" case does trigger the detached head state with its
warnings. But the other alternative I showed does not:

	git fetch git://... branch-name:branch-name

Here, of course the user can use:

	git checkout branch-name

and not ever enter the "detached HEAD" state at all. So with this
usage the discussion about where and when to warn becomes moot.

But this still isn't satisfying to me as something to offer users, as
I'd really like them to be able to just "git pull" to follow
subsequent things I commit to the branch. But for that the user would
still need a bunch of configuration setup.

So it does come around to the fact that I'd like it to be easier for a
user to get all the configuration setup for a local branch that knows
which remote-tracking branch its associated with, (and this whether or
not the remote-tracking branch was configured as part of the original
clone or not).

-Carl
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] tone down the detached head warning, Nicolas Pitre, (Wed Jan 31, 3:10 pm)
Re: [PATCH] tone down the detached head warning, Jeff King, (Wed Jan 31, 7:19 pm)
Re: [PATCH] tone down the detached head warning, Nicolas Pitre, (Wed Jan 31, 8:11 pm)
Re: [PATCH] tone down the detached head warning, Jeff King, (Wed Jan 31, 11:00 pm)
Re: [PATCH] tone down the detached head warning, Carl Worth, (Wed Jan 31, 7:54 pm)