git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "norbert.nemec" <norbert.nemec@native-instruments.de>
To: git@vger.kernel.org
Subject: Re: How to find and analyze bad merges?
Date: Thu, 02 Feb 2012 13:16:07 +0100	[thread overview]
Message-ID: <jgduqg$p9f$1@dough.gmane.org> (raw)
In-Reply-To: <20120202120340.GA25190@burratino>

Am 02.02.12 13:03, schrieb Jonathan Nieder:
> David Barr wrote:
>
>> Do the -c --cc or -m flags for git log help in this case?
>> They alter the way merge diffs are presented, as described under Diff Formatting
>> in the git-log(1)  man page.
>
> I suspect Norbert was running into history simplification, so the --full-history
> flag would be the relevant one.

Not quite.

As far as I understand it, history simplification hides the whole branch 
if its changes did not end up in the current branch.

When I tried it out, the --full-history prevented hiding the 
bugfix-commit itself, but it did not show the critical merge commit in 
the log.

> See the thread [1] for a few relevant side-notes.
 >
 > [1] http://thread.gmane.org/gmane.comp.version-control.git/188904

As I understand this thread, the user only requested all commits that 
"modify a file". Our merge-commit strictly speaking did not modify the 
file but simply kept one of the versions, completely swamping all 
modifications from one branch. Exactly the case that is still not 
covered by --full-history.

  reply	other threads:[~2012-02-02 12:16 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-02  8:10 How to find and analyze bad merges? norbert.nemec
2012-02-02  8:16 ` Junio C Hamano
2012-02-02  9:01   ` norbert.nemec
2012-02-02 20:09     ` Junio C Hamano
2012-02-02 10:05 ` norbert.nemec
     [not found]   ` <87haz97c2k.fsf@thomas.inf.ethz.ch>
2012-02-02 11:17     ` Norbert Nemec
2012-02-02 11:41       ` David Barr
2012-02-02 12:03         ` Jonathan Nieder
2012-02-02 12:16           ` norbert.nemec [this message]
2012-02-02 15:09             ` Neal Groothuis
2012-02-02 12:10         ` norbert.nemec

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='jgduqg$p9f$1@dough.gmane.org' \
    --to=norbert.nemec@native-instruments.de \
    --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).