git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Marius Storm-Olsen <mstormo@gmail.com>
To: Git Mailing List <git@vger.kernel.org>
Subject: git-diff/git-diff-files: Turn off copy-detection?
Date: Tue, 29 Sep 2009 14:50:14 +0200	[thread overview]
Message-ID: <4AC20286.6020108@gmail.com> (raw)

Hi,

I'm trying to diff the header files between two version of a repo, to 
do an ABI review, but get too many diffs from 'similar copy', which 
makes the review hard. These are generally unrelated files, but due to 
large copyright header, and spaces, the actual changes are not large 
enough to be detected as new files. I'd therefore like to turn off 
copy detection completely, or at least set some kind of threshold 
before a file is considered a copy in the diff.

Doing a 'git diff origin/4.5..origin/4.6 -- src/corelib' gives me this 
excerpt:

diff --git origin/4.5/src/corelib/tools/qtimeline.h 
origin/4.6/src/corelib/animation/qabstractanimation.h
similarity index 58%
...
diff --git origin/4.5/src/corelib/kernel/qsystemsemaphore.h 
origin/4.6/src/corelib/animation/qanimationgroup.h
similarity index 63%
...
diff --git origin/4.5/src/corelib/kernel/qsystemsemaphore.h 
origin/4.6/src/corelib/animation/qparallelanimationgroup.h
similarity index 63%
...
diff --git origin/4.5/src/corelib/io/qresource_iterator_p.h 
origin/4.6/src/corelib/animation/qpauseanimation.h
similarity index 68%
...
diff --git origin/4.5/src/corelib/kernel/qsystemsemaphore.h 
origin/4.6/src/corelib/animation/qpropertyanimation.h
similarity index 60%
...
diff --git origin/4.5/src/corelib/kernel/qsystemsemaphore.h 
origin/4.6/src/corelib/animation/qsequentialanimationgroup.h
similarity index 55%
...
diff --git origin/4.5/src/corelib/animation/qvariantanimation.h 
origin/4.6/src/corelib/animation/qvariantanimation.h
new file mode 100644
...
diff --git origin/4.5/src/corelib/arch/qatomic_arch.h 
origin/4.6/src/corelib/arch/qatomic_arch.h
index 52ab101..548a5e9 100644

So, not even related. They should really had shown up as new files, 
IMO. (I guess git stores the creation of the file as a delta of the 
indicated 'copy', but in this case I don't care about that :)

Thanks!

--
.marius

             reply	other threads:[~2009-09-29 12:50 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-29 12:50 Marius Storm-Olsen [this message]
2009-09-29 13:40 ` git-diff/git-diff-files: Turn off copy-detection? Johannes Sixt
2009-09-29 14:24   ` Marius Storm-Olsen
2009-09-29 15:05     ` Miklos Vajna
2009-09-29 13:42 ` Miklos Vajna

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=4AC20286.6020108@gmail.com \
    --to=mstormo@gmail.com \
    --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).