From: David Kastrup <dak@gnu.org>
To: Junio C Hamano <gitster@pobox.com>
Cc: git@vger.kernel.org
Subject: Re: git-commit --amend -m "..." complains?!?
Date: Wed, 01 Aug 2007 21:23:18 +0200 [thread overview]
Message-ID: <85myxbrrdl.fsf@lola.goethe.zz> (raw)
In-Reply-To: <7v8x8v5g2z.fsf@assigned-by-dhcp.cox.net> (Junio C. Hamano's message of "Wed\, 01 Aug 2007 10\:18\:12 -0700")
Junio C Hamano <gitster@pobox.com> writes:
> David Kastrup <dak@gnu.org> writes:
>
>> I get
>>
>> Option -m cannot be combined with -c/-C/-F/--amend.
>>
>> but that makes no sense: of course there is ample reason for providing
>> an amended commit message on the command line. -c, -C and -F indeed
>> all provide an alternative commit message, but --amend doesn't.
>
> The option --amend is about "reusing the original commit message
> and make amending edit on top".
Uh no. From the man page:
--amend
Used to amend the tip of the current branch. Prepare the
tree object you would want to replace the latest commit
as usual (this includes the usual -i/-o and explicit
paths), and the commit log editor is seeded with the
commit message from the tip of the current branch. The
commit you create replaces the current tip -- if it was a
merge, it will have the parents of the current tip as
parents -- so the current top commit is discarded.
It is a rough equivalent for:
$ git reset --soft HEAD^
$ ... do something else to come up with the right tree ...
$ git commit -c ORIG_HEAD
but can be used to amend a merge commit.
The --amend is not, according to the manual page, there to amend the
commit message, but primarily to amend the commit.
If one does not want to amend the message, this is easily done with -C
HEAD. If one wants to amend the message but not by editing, one is
plain out of luck.
-C, -F and -m are all logically exclusive (and -C -e is the same as
-c). But --amend seems completely orthogonal to me with regard to the
commit message: it just has a different seed from the normal commit
message edit. You can override the seed with a normal commit using -c
-C -F and -m. Why not with --amend?
> If you are restarting the message from scratch, --amend does not
> make much sense.
It amends the _commit_. You can edit files, add them, and commit
additional changes. If you are using a dumb terminal (or a system
with nonworking VISUAL over the link in question), you don't _want_ an
editor called up.
> You can do:
>
> $ git reset HEAD^
> $ git commit -m "blah"
>
> if you do not want to reuse the commit message.
You can pretty much _always_ avoid --amend in a similar manner, but
why would you? It is convenient.
--
David Kastrup, Kriemhildstr. 15, 44793 Bochum
next prev parent reply other threads:[~2007-08-01 19:24 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-01 14:25 git-commit --amend -m "..." complains?!? David Kastrup
2007-08-01 17:18 ` Junio C Hamano
2007-08-01 19:23 ` David Kastrup [this message]
2007-08-01 19:52 ` Junio C Hamano
2007-08-01 20:45 ` David Kastrup
2007-08-01 20:33 ` [PATCH] git-commit.sh: Permit the --amend message to be given with -m/-c/-C/-F David Kastrup
2007-08-02 1:03 ` Junio C Hamano
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=85myxbrrdl.fsf@lola.goethe.zz \
--to=dak@gnu.org \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.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