linux-kernel - Re: Would it be better for git-commits-head to have -M -C diffs?

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 14 Aug 2008 10:15:41 -0700
From:	Randy Dunlap <randy.dunlap@...cle.com>
To:	James Cloos <cloos@...loos.com>
Cc:	linux-kernel@...r.kernel.org, David Woodhouse <dwmw2@...radead.org>
Subject: Re: Would it be better for git-commits-head to have -M -C diffs?

On Thu, 14 Aug 2008 12:56:18 -0400 James Cloos wrote:

> Would anyone else prefer to see the posts on the commits list use git's
> copy and move detection?
> 
> Git show (used by the feed-mail-list script) does support -M and -C.
> 
> It does affect the stat section.  The recent commit 758db3f21187 would
> have had its stat changed from looking like:
> 
>  arch/h8300/include/asm/Kbuild              |    1 +
>  arch/h8300/include/asm/a.out.h             |   20 ++
>  arch/h8300/include/asm/atomic.h            |  144 +++++++++++
>                         .
>                         .
>                         .
>  include/asm-h8300/Kbuild                   |    1 -
>  include/asm-h8300/a.out.h                  |   20 --
>  include/asm-h8300/atomic.h                 |  144 -----------
>                         .
>                         .
>                         .
> 
> to looking like:
> 
>  .../asm-h8300 => arch/h8300/include/asm}/Kbuild    |    0 
>  .../asm-h8300 => arch/h8300/include/asm}/a.out.h   |    0 
>  .../asm-h8300 => arch/h8300/include/asm}/atomic.h  |    0 
>  .../asm-h8300 => arch/h8300/include/asm}/auxvec.h  |    0 
>  .../asm-h8300 => arch/h8300/include/asm}/bitops.h  |    0 
>  .../h8300/include/asm}/bootinfo.h                  |    0 
>                         .
>                         .
>                         .
> 
> but would have been 512 lines and 24271 octets instead of 10623 lines
> and 291634 octets.
> 
> I think it would also be useful were the From: header of each message
> set to the commit's Author.  I run each commit through a script to do
> that locally, but the archives would also benefit from that.


or the git-commits list could use what Linus has requested to be used
in pull requests:

Please use "git diff -M --stat --summary" to generate the diffstat:
the -M enables rename detection, and the summary enables a summary of
new/deleted or renamed files.


---
~Randy
Linux Plumbers Conference, 17-19 September 2008, Portland, Oregon USA
http://linuxplumbersconf.org/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/