Re: [PATCH] blame tests - add test for blame output after merge

On 05/13/2009 10:30 AM, Alan Wood wrote:
> Hi Lieven,
> Thanks for committing the test. Nice to see how it should be done.

Not all of those changes where really needed, just wanted to make the
minimal test to reproduce the issue.

In general we try to avoid cd'ing into the working copy directory to
execute the test commands, 'cause this breaks running multiple tests in
parallel.

I also changed the first line of the 'mu' file. This file is written
with LF line endings in the greek tree, so if you on Windows recreate
the file with the same line of text, it will still be marked changed
because you write a CRLF. On linux, the new file will also have LF line
endings so the line will not be marked 'changed'. That's impossible for
you to know, just adding it fyi.

For the rest the test was very good. Thanks!

Lieven

> Now an XPASS on my system so just have to get some movement on the patch for
> the fix now. Hopefully wont need as many changes as the test did.
> http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1875446
> Alan

See the other mail thread.

Lieven

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2240581
Received on 2009-05-13 19:42:05 CEST