From: "Shawn O. Pearce" <spearce@spearce.org>
To: Junio C Hamano <gitster@pobox.com>
Cc: Robin Rosenberg <robin.rosenberg.lists@dewire.com>,
git@vger.kernel.org,
Johannes Schindelin <Johannes.Schindelin@gmx.de>
Subject: Re: [PATCH] Completion: Hint whether amend is the right thing to do.
Date: Wed, 28 May 2008 19:16:22 -0400 [thread overview]
Message-ID: <20080528231622.GL30245@spearce.org> (raw)
In-Reply-To: <7v63sywhkr.fsf@gitster.siamese.dyndns.org>
Junio C Hamano <gitster@pobox.com> wrote:
> Robin Rosenberg <robin.rosenberg.lists@dewire.com> writes:
>
> > When you get a conflict during interactive rebase the next
> > thing to do is not to amend, but to continue the rebase.
> > With this change you get a reminder in the prompt.
...
> Does this mean that it would help reduce mistakes if "git commit --amend"
> noticed the presense of .dotest-merge but absense of .dotest-merge/amend
> and refused to operate, I have to wonder?
Yea, even though rebase and commit are two different tools we
don't want the user to amend in the middle of a failed conflict
from rebase. Adding the safety valve to commit --amend makes
more sense to me than teaching bash to show you the state. Not
all of our users use bash. Not all of our bash using users use
the bash completion package, or the _git_ps1 function we ship.
Etc.
I'd rather see the amend safety net than the bash prompt change.
--
Shawn.
next prev parent reply other threads:[~2008-05-28 23:17 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-28 14:08 [PATCH] Completion: Hint whether amend is the right thing to do Robin Rosenberg
2008-05-28 15:53 ` Junio C Hamano
2008-05-28 23:16 ` Shawn O. Pearce [this message]
2008-05-28 23:33 ` Robin Rosenberg
2008-05-28 23:39 ` Junio C Hamano
2008-05-28 23:45 ` Shawn O. Pearce
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=20080528231622.GL30245@spearce.org \
--to=spearce@spearce.org \
--cc=Johannes.Schindelin@gmx.de \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=robin.rosenberg.lists@dewire.com \
/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).