All of lore.kernel.org
 help / color / mirror / Atom feed
From: Aghiles <aghilesk@gmail.com>
To: Jeff King <peff@peff.net>
Cc: Matthieu Moy <Matthieu.Moy@imag.fr>,
	Nicolas Sebrecht <nicolas.s.dev@gmx.fr>,
	Thomas Rast <trast@student.ethz.ch>,
	git list <git@vger.kernel.org>
Subject: Re: git pull suggestion
Date: Sun, 11 Apr 2010 02:01:18 -0400	[thread overview]
Message-ID: <m2v3abd05a91004102301i95bf7091ib2bd9da5e8a208c1@mail.gmail.com> (raw)
In-Reply-To: <20100410043535.GA22481@coredump.intra.peff.net>

King <peff@peff.net> writes:
> I think we would do better to tell the user about stash there, so they
> can do it themselves. Then they know where their changes went and how to
> get them back. Since v1.6.5.5, this error message now says:
>
>  Your local changes to '%s' would be overwritten by merge.  Aborting.
>  Please, commit your changes or stash them before you can merge.
>
> What version of git are you using? If you (or others you are helping)
> saw that message and it wasn't helpful, do you have any suggestions for
> how to improve it?

Yes we have the latest version and we do see this message. This helps
a bit. Although for people used to CVS/CVN the "stash" is yet another thing
to learn. There is also a high probability for new users to see this message
very early when using git and the question is always the same: why can't git
just merge with my files and show me the conflict?

(There was also some usability issues with the "stash", I remember people
loosing _untracked_ files but I am not sure if that was PEBKAC. First versions
of stash had a very friendly syntax that punished you by obliterating your files
if you made a typo and some are still traumatized.)

Simply put: in git, your working directory is a second class citizen and git
doesn't want to deal with it. Fundamentally, this is in a collision course with
what some users think about their work in progress.

I started a similar discussion had a couple years ago:
http://lists-archives.org/git/635926-git-pull-opinion.html

Back then, I was certain that 'git pull' should have an option to mix with a
dirty tree but now, after a couple years of using the tool, I am not certain
anymore. I am just reporting the biggest frustrations I see with new users.

Thanks,

  -- aghiles

  parent reply	other threads:[~2010-04-11  6:01 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-07 23:17 git pull suggestion Aghiles
2010-04-08 15:54 ` Thomas Rast
2010-04-08 19:33   ` Aghiles
2010-04-08 23:11     ` Nicolas Sebrecht
2010-04-09  3:06       ` Aghiles
2010-04-09  3:49         ` Jeff King
2010-04-09 19:33           ` Aghiles
2010-04-10  4:35             ` Jeff King
2010-04-10  4:40               ` Junio C Hamano
2010-04-11  6:01               ` Aghiles [this message]
2010-04-11  7:37                 ` Junio C Hamano
2010-04-11 16:33                   ` Matthieu Moy
2010-04-12 20:18                   ` Aghiles
2010-04-12 21:35                     ` Junio C Hamano
2010-04-09 20:54           ` Aghiles

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=m2v3abd05a91004102301i95bf7091ib2bd9da5e8a208c1@mail.gmail.com \
    --to=aghilesk@gmail.com \
    --cc=Matthieu.Moy@imag.fr \
    --cc=git@vger.kernel.org \
    --cc=nicolas.s.dev@gmx.fr \
    --cc=peff@peff.net \
    --cc=trast@student.ethz.ch \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.