git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Nicolas Pitre <nico@cam.org>
To: Carl Worth <cworth@cworth.org>
Cc: Alan Chandler <alan@chandlerfamily.org.uk>,
	git@vger.kernel.org, Junio C Hamano <junkio@cox.net>
Subject: Re: [PATCH] make 'git add' a first class user friendly interface to the index
Date: Sat, 02 Dec 2006 23:22:11 -0500 (EST)	[thread overview]
Message-ID: <Pine.LNX.4.64.0612022305100.2630@xanadu.home> (raw)
In-Reply-To: <87psb22qgu.wl%cworth@cworth.org>

On Sat, 2 Dec 2006, Carl Worth wrote:

> If git's model imposes the requirement, "we should first teach one
> thing, then move on to teach a subsequent thing", it would be just
> that much nicer if the commands themselves could help us do that,
> (because the default would do the thing they would need first, and
> then the user has to explicitly do _something_ else to get the
> subsequent thing).

Since I've been thinking about this issue I've come to the conclusion 
that making commit -a the default for commit is not a good thing.

Why? Because we really want newbies to be tricked into using the index.

And teaching about the different ways to update the index in the 
tutorial right after the first commit example is IMHO the best thing to 
do.

Making commit -a the default would make it possible for newbies to 
get along for a long while ignoring the usage model of git and that is 
bad.

I think the idea is really to make "git commit" with a clean index more 
clueful to the user.  Right now it only says "use git-update-index to 
mark for commit" which is really not that helpful, and actually the 
point of failure with the example newbie problem you pointed out.

There is a compromise to reach.  Sure the _user_ needs a proper model to 
use the tool without being bothered with technical implementation or 
architecture details.  But we still need newbies to get into the git 
model nevertheless, and having a default for git-commit geared towards 
making it bump free for new users is not the way to go I think.  The 
"nothing to commit" message needs to be way more helpful with better 
guidance and the git-commit default behavior should be overcome.



  parent reply	other threads:[~2006-12-03  4:22 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-01 20:06 [PATCH] make 'git add' a first class user friendly interface to the index Nicolas Pitre
2006-12-01 22:31 ` Junio C Hamano
2006-12-02  0:18   ` Alan Chandler
2006-12-02  2:01     ` Nicolas Pitre
2006-12-02  3:05   ` Nicolas Pitre
2006-12-02  6:54   ` Carl Worth
2006-12-02  7:54     ` Junio C Hamano
2006-12-02  9:06       ` Carl Worth
2006-12-02 10:11       ` Jakub Narebski
2006-12-02 14:51         ` Han-Wen Nienhuys
2006-12-02  8:28     ` Alan Chandler
2006-12-02 16:49       ` Carl Worth
2006-12-02 17:12         ` Jakub Narebski
2006-12-02 18:05         ` Alan Chandler
2006-12-03  4:04           ` Nicolas Pitre
2006-12-03  4:22         ` Nicolas Pitre [this message]
2006-12-03  4:34       ` Nicolas Pitre
2006-12-02  9:52   ` Jakub Narebski
2006-12-03  5:03   ` Nicolas Pitre
2006-12-03  5:33 ` [PATCH v2] " Nicolas Pitre
2006-12-03  9:16   ` Alan Chandler

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Pine.LNX.4.64.0612022305100.2630@xanadu.home \
    --to=nico@cam.org \
    --cc=alan@chandlerfamily.org.uk \
    --cc=cworth@cworth.org \
    --cc=git@vger.kernel.org \
    --cc=junkio@cox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).