From: Jonathan Nieder <jrnieder@gmail.com>
To: Ramkumar Ramachandra <artagnon@gmail.com>
Cc: Git List <git@vger.kernel.org>,
Junio C Hamano <gitster@pobox.com>, Jeff King <peff@peff.net>,
Daniel Barkalow <barkalow@iabervon.org>,
Christian Couder <chriscool@tuxfamily.org>
Subject: Re: [RFC PATCH v2] revert: Implement --abort processing
Date: Sun, 12 Jun 2011 17:12:17 -0500 [thread overview]
Message-ID: <20110612221217.GA2789@elie> (raw)
In-Reply-To: <BANLkTi=gYSJgG-Yu-+-5wPetpb0-AW=X+g@mail.gmail.com>
Ramkumar Ramachandra wrote:
> Frankly, I'm still trying to understand how other people work -- I
> don't use the porcelain much, and I dislike anything but the most
> minimalistic automation. I didn't even like the change to cherry-pick
> where you can simply "git commit" after resolving a conflict; I still
> prefer and use the more explicit "git commit -c" after removing the
> CHERRY_PICK_HEAD. Also, I *always* use rebase with --onto
I don't think that should stop you from thinking about how new
facilities help or interfere with work at all. You use magit, right?
It automates all kinds of things. And while each person is going to
use tools in different ways, that hasn't kept people from getting
things done in the past.
If you are thinking "I would never use 'git cherry-pick --abort' --- I
would just look in the reflog for a commit to 'reset --hard' to", then
you are *done*. Just document it, make sure the reflog has useful
content to help out, and wait until someone complains and adds a
shortcut they like.
Addendum: Personally I was happy about "git commit" that implicitly
takes the commit message from CHERRY_PICK_HEAD because it adds a
Conflicts:
Makefile
that I can use as a template for a message about the nature of the
conflict and how I fixed it up. As a side note, I'm curious about
why you end up needing to remove the CHERRY_PICK_HEAD. Is "git commit
-c interesting-patch" misbehaving somehow? (It should ignore the
CHERRY_PICK_HEAD entirely.)
next prev parent reply other threads:[~2011-06-12 22:12 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-11 6:36 [RFC PATCH v2] revert: Implement --abort processing Ramkumar Ramachandra
2011-06-11 11:22 ` Jonathan Nieder
2011-06-12 12:09 ` Ramkumar Ramachandra
2011-06-12 12:21 ` Jonathan Nieder
2011-06-12 12:51 ` Ramkumar Ramachandra
2011-06-12 22:12 ` Jonathan Nieder [this message]
2011-06-13 14:55 ` Ramkumar Ramachandra
2011-06-13 20:28 ` Jonathan Nieder
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=20110612221217.GA2789@elie \
--to=jrnieder@gmail.com \
--cc=artagnon@gmail.com \
--cc=barkalow@iabervon.org \
--cc=chriscool@tuxfamily.org \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=peff@peff.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).