From: Linus Torvalds <torvalds@linux-foundation.org>
To: Greg KH <greg@kroah.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
linux-kernel@vger.kernel.org, devel@driverdev.osuosl.org
Subject: Re: [GIT PATCH] STAGING patches for 2.6.31-git
Date: Wed, 16 Sep 2009 08:26:14 -0700 (PDT) [thread overview]
Message-ID: <alpine.LFD.2.01.0909160813400.4950@localhost.localdomain> (raw)
In-Reply-To: <20090915190520.GA834@kroah.com>
On Tue, 15 Sep 2009, Greg KH wrote:
>
> Overall, we almost broke even with our cleanups and removals for what
> was done here:
> 1267 files changed, 297615 insertions(+), 214568 deletions(-)
Actually, it's
1253 files changed, 287789 insertions(+), 209389 deletions(-)
but there are so many changed files that the default rename detection
limit kicks in, and your git will have only looked at 100% identical
renames. So you missed things like:
rename drivers/staging/{rt2860 => rt3090}/link_list.h (99%)
copy drivers/staging/{rt2870/md4.h => rt3090/mlme_ex.h} (56%)
copy drivers/staging/{rt2860/md4.h => rt3090/mlme_ex_def.h} (67%)
copy drivers/staging/{rt2860/md4.h => rt3090/netif_block.h} (72%)
rename drivers/staging/{rt2870/md4.h => rt3090/rt33xx.h} (79%)
...
Now, admittedly the 'copy' ones are dubious: it's picking up on the
f*cking Ralink GPL copyright template, which is the bulk of those files.
But that in itself is interesting information, even if I personally find
mindless copyright templates very annoying.
Anyway, extended rename detection (ie the non-exact one that can't just
look at the SHA1's and say "those are identical") gets disabled if your
rename matrix is large (cost function: n*m where 'n' is number of added
files, and 'm' is number of deleted ones), which triggers for your staging
tree because you had both deletions and creations.
The default git limit is set fairly low, because it's set for small
machines with limited amounts of memory in particular. I suspect you might
want to add
[diff]
renamelimit=0
to your ~/.gitconfig, since you probably don't use particularly weak
machines, and that will disable the limiting. Then you'd get the full
renames in all their glory.
Without it, you'll see just:
copy drivers/staging/{rtl8192su => rtl8192e}/dot11d.h (100%)
rename drivers/staging/{rtl8192su => rtl8192e/ieee80211}/dot11d.h (100%)
rename drivers/staging/{rtl8192su => rtl8192e}/ieee80211/rtl_crypto.h (100%)
rename drivers/staging/{rtl8187se => rtl8192e}/ieee80211_crypt.h (100%)
while with it, you'll see examples of lots of cases of header files where
there is more copyright header than actual content ;(
Linus
next prev parent reply other threads:[~2009-09-16 15:26 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-09-15 19:05 [GIT PATCH] STAGING patches for 2.6.31-git Greg KH
2009-09-16 15:26 ` Linus Torvalds [this message]
2009-09-16 15:34 ` Greg KH
2009-09-18 15:47 ` Dave Taht
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=alpine.LFD.2.01.0909160813400.4950@localhost.localdomain \
--to=torvalds@linux-foundation.org \
--cc=akpm@linux-foundation.org \
--cc=devel@driverdev.osuosl.org \
--cc=greg@kroah.com \
--cc=linux-kernel@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).