On Thu, Nov 2, 2017 at 12:23 AM, Andreas Schwab <schwab@xxxxxxxxxxxxxx> wrote: > On Nov 01 2017, Johannes Schindelin <Johannes.Schindelin@xxxxxx> wrote: > >> Sure, but it is still a tricky thing. Imagine >> >> - A1 - B1 - A2 - B2 - B3 >> >> where all the B* commits have the blob. Do you really want to report B1 >> rather than B2 as the commit introducing the blob? (I would prefer B2...) > > What if B3 renames or copies the blob? > > Andreas. With the current proposed patch you'd find B3, and then use the diff machinery to digg deeper from there (renames/copies ought to be easy to detect already?) So with a copy B3 might be a better start than B1, as starting from B1 you would not find B3 easily. For a rename, I would think a reverse log/blame on B1:path may help. With that said, I think I'll just reroll the series with the current logic fixing the other minor issues that were brought up as B3 seems to be the most versatile (though not optimal) answer for many use cases. Thanks for that thought, Stefan
