Re: [PATCH v2 1/2] Add test to describe expectation of blame --reverse with branched history
On Wed, Oct 21, 2015 at 09:19:45PM -0700, Junio C Hamano wrote:
Show 14 quoted lines
> Let me try.
>
> (1) for merges, an edge with '*' denotes the one to the first
> parent.
> (2) a commit that touches file.t are in capital
>
> c1---a3
> / *
> / /
> A0---a1--*a2
> \ /
> b1---B2
>
> Did I draw it correctly?
Show 9 quoted lines
>> +test_expect_failure 'blame --reverse finds B1, not C1' '
>> + git blame --porcelain --reverse A0..A3 -- file.t >actual_full &&
>> + head -1 <actual_full | sed -e "sX .*XX" >actual &&
>
> "head -n 1" is more POSIXy way to spell this.
>
> Don't get cute with sXXX in sed script when the usual s/// suffices;
> the only effect of the cuteness is to waste readers' time, who have
> to wonder if there is something interesting going on.
Ok. Will fix it and add the graph.