Re: [RFC/PATCH] add diff-pairs tool

2016-12-01 Thread Jeff King
On Thu, Dec 01, 2016 at 02:22:47PM -0800, Junio C Hamano wrote: > Jeff King writes: > > >> It took me a while to dig it up because the topic is so old, but > >> > >> https://public-inbox.org/git/pine.lnx.4.58.0504251832480.18...@ppc970.osdl.org/ > >> > >> is the thread I had in mind. The idea

Re: [RFC/PATCH] add diff-pairs tool

2016-12-01 Thread Junio C Hamano
Jeff King writes: >> It took me a while to dig it up because the topic is so old, but >> >> https://public-inbox.org/git/pine.lnx.4.58.0504251832480.18...@ppc970.osdl.org/ >> >> is the thread I had in mind. The idea of rename detection followed >> soon afterwards. > > Thanks for an interesting

Re: [RFC/PATCH] add diff-pairs tool

2016-12-01 Thread Jeff King
On Thu, Dec 01, 2016 at 12:58:21PM -0800, Junio C Hamano wrote: > Jeff King writes: > > > On Thu, Dec 01, 2016 at 12:52:05PM -0800, Junio C Hamano wrote: > > > >> Jeff King writes: > >> > >> > This takes the output of `diff-tree -z --raw` and feeds it > >> > back to the later stages of the dif

Re: [RFC/PATCH] add diff-pairs tool

2016-12-01 Thread Junio C Hamano
Junio C Hamano writes: > It took me a while to dig it up because the topic is so old, but > > https://public-inbox.org/git/pine.lnx.4.58.0504251832480.18...@ppc970.osdl.org/ > > is the thread I had in mind. The idea of rename detection followed > soon afterwards. ... which was this one: https:

Re: [RFC/PATCH] add diff-pairs tool

2016-12-01 Thread Junio C Hamano
Jeff King writes: > On Thu, Dec 01, 2016 at 12:52:05PM -0800, Junio C Hamano wrote: > >> Jeff King writes: >> >> > This takes the output of `diff-tree -z --raw` and feeds it >> > back to the later stages of the diff machinery to produce >> > diffs in other formats. >> >> A full circle. This r

Re: [RFC/PATCH] add diff-pairs tool

2016-12-01 Thread Junio C Hamano
Jeff King writes: > This takes the output of `diff-tree -z --raw` and feeds it > back to the later stages of the diff machinery to produce > diffs in other formats. A full circle. This reminds me of the experiment done more than 10 years ago at the beginning of the "diffcore transformations" de

Re: [RFC/PATCH] add diff-pairs tool

2016-12-01 Thread Jeff King
On Thu, Dec 01, 2016 at 12:52:05PM -0800, Junio C Hamano wrote: > Jeff King writes: > > > This takes the output of `diff-tree -z --raw` and feeds it > > back to the later stages of the diff machinery to produce > > diffs in other formats. > > A full circle. This reminds me of the experiment do

[RFC/PATCH] add diff-pairs tool

2016-12-01 Thread Jeff King
This is a small tool I cooked up for splitting the tree-diff out from the generation of the patch text, and doing them in separate processes. It's a little more expensive than doing it all in one process (besides the process/pipe overhead, we may load blob content in the tree diff for rename detect