From: Jay Soffian <jaysoffian@gmail.com>
To: Michael J Gruber <git@drmicha.warpmail.net>
Cc: git <git@vger.kernel.org>
Subject: Re: Not possible to see combined diff before committing a merge?
Date: Thu, 18 Mar 2010 11:21:13 -0400 [thread overview]
Message-ID: <76718491003180821x777df87j245d134e32ec90dc@mail.gmail.com> (raw)
In-Reply-To: <4BA1F880.2090100@drmicha.warpmail.net>
On Thu, Mar 18, 2010 at 5:55 AM, Michael J Gruber
<git@drmicha.warpmail.net> wrote:
> If there are no conflicted files left you could do a throw-away commit
> with a temp index, but that looks ugly. Do we have a way of specifying a
> 3-way diff for some git diff --cc incarnation, i.e. pretending two
> commits to be the parents and a third (or index) to be the merge result?
It would seem that diff-index should take two <tree-ish>'s (it only
takes one) to support this.
> Junio would probably say that by adding the merge resolution you declare
> that you're not interested in that part of the merge any more ;)
The use case is that I use mergetool when I have merges with very
large conflicted regions that are too difficult to visualize using the
merge and/or diff3 conflict style. But because these conflict
resolutions are so difficult, I often want to look at the merge
resolution in combined-diff style before committing.
The way I would have to do this today is:
- git mergetool <path>
- save the result in my visual merge tool and quit
- in a separate shell, use git diff to see the 3-way diff
- in the first shell, answer mergetool's prompt that it's okay to use the result
Hmm, maybe there should be an option for whether mergetool
automatically add's the result for you.
Anyway, what I've noticed is that I've become gun shy about using "git
add" myself during conflict resolution because as soon as I do, I lose
some functionality compared to the unmerged state.
j.
prev parent reply other threads:[~2010-03-18 15:21 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-18 6:21 Not possible to see combined diff before committing a merge? Jay Soffian
2010-03-18 9:55 ` Michael J Gruber
2010-03-18 15:21 ` Jay Soffian [this message]
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=76718491003180821x777df87j245d134e32ec90dc@mail.gmail.com \
--to=jaysoffian@gmail.com \
--cc=git@drmicha.warpmail.net \
--cc=git@vger.kernel.org \
/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).