Vitor Antunes <vitor....@gmail.com> writes:

> File file11 is copied from file2 and diff-tree correctly reports this file as
> its the source, but the test expression was checking for file10 instead (which
> was a file that also originated from file2). It is possible that the diff-tree
> algorithm was updated in recent versions, which resulted in this mismatch in
> behavior.
>
> Signed-off-by: Vitor Antunes <vitor....@gmail.com>

Pete, these tests blame to your 9b6513ac (git p4 test: split up big
t9800 test, 2012-06-27).  I presume that you tested the result of
this splitting, but do you happen to know if we did something to
cause the test to break recently?

> ---
>  t/t9814-git-p4-rename.sh |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/t/t9814-git-p4-rename.sh b/t/t9814-git-p4-rename.sh
> index 24008ff..018f01d 100755
> --- a/t/t9814-git-p4-rename.sh
> +++ b/t/t9814-git-p4-rename.sh
> @@ -156,14 +156,14 @@ test_expect_success 'detect copies' '
>               git diff-tree -r -C HEAD &&
>               git p4 submit &&
>               p4 filelog //depot/file10 &&
> -             p4 filelog //depot/file10 | grep -q "branch from //depot/file" 
> &&
> +             p4 filelog //depot/file10 | grep -q "branch from //depot/file2" 
> &&
>  
>               cp file2 file11 &&
>               git add file11 &&
>               git commit -a -m "Copy file2 to file11" &&
>               git diff-tree -r -C --find-copies-harder HEAD &&
>               src=$(git diff-tree -r -C --find-copies-harder HEAD | sed 1d | 
> cut -f2) &&
> -             test "$src" = file10 &&
> +             test "$src" = file2 &&
>               git config git-p4.detectCopiesHarder true &&
>               git p4 submit &&
>               p4 filelog //depot/file11 &&
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to