You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Paul Burba <pt...@gmail.com> on 2009/06/25 14:45:22 UTC

Re: svn commit: r38181 - in trunk/subversion: libsvn_client tests/cmdline

Neels,

I fixed in r31198.

Paul

On Thu, Jun 25, 2009 at 10:08 AM, Neels Janosch Hofmeyr<ne...@elego.de> wrote:
> Hey Bert,
>
> damn, sorry for breaking that ... I was trying to be "too compatible",
> probably, by adding os.pathsep instead of slash...
>
> Unfortunately I have no time to fix it right now, won't be back until
> monday. If it's really annoying, you can remove the test for now, and I'll
> re-add it later.
>
> Thanks
> ~Neels
>
>
> Bert Huijben wrote:
>>> -----Original Message-----
>>> From: Neels Janosch Hofmeyr [mailto:neels@elego.de]
>>> Sent: woensdag 24 juni 2009 1:21
>>> To: svn@subversion.tigris.org
>>> Subject: svn commit: r38181 - in trunk/subversion: libsvn_client
>>> tests/cmdline
>>>
>>> Author: neels
>>> Date: Tue Jun 23 16:20:37 2009
>>> New Revision: 38181
>>>
>>> Log:
>>> * subversion/libsvn_client/merge.c
>>>   (merge_dir_added): Use special case function tree_conflict_on_add().
>>>
>>> * subversion/libsvn_client/repos_diff.c
>>>   (add_directory, close_file): Skip notify on tree-conflicts during a
>>> replace.
>>>     Add comment and some whitespace.
>>>
>>> * subversion/tests/cmdline/merge_tests.py
>>>   (merge_replace_causes_tree_conflict): Inflate the test to use all of
>>>     the cases (dir|file) replaces (dir|file).
>>>   (test_list): Pass merge_replace_causes_tree_conflict().
>>
>> This test is now failing on the Windows buildbot while it passed before.
>>
>> See http://crest.ics.uci.edu/buildbot/waterfall
>>
>> EXPECTED STDOUT (unordered):
>>
>> D         svn-test-work\working_copies\merge_tests-132\branch\mu
>> D         svn-test-work\working_copies\merge_tests-132\branch\B\E/alpha
>> D         svn-test-work\working_copies\merge_tests-132\branch\B\E/beta
>> D         svn-test-work\working_copies\merge_tests-132\branch\B\E
>> D         svn-test-work\working_copies\merge_tests-132\branch\D\G\pi
>> D         svn-test-work\working_copies\merge_tests-132\branch\D\H/chi
>> D         svn-test-work\working_copies\merge_tests-132\branch\D\H/omega
>> D         svn-test-work\working_copies\merge_tests-132\branch\D\H/psi
>> D         svn-test-work\working_copies\merge_tests-132\branch\D\H
>>
>> ACTUAL STDOUT:
>>
>> D         svn-test-work\working_copies\merge_tests-132\branch\mu
>> D         svn-test-work\working_copies\merge_tests-132\branch\B\E\alpha
>> D         svn-test-work\working_copies\merge_tests-132\branch\B\E\beta
>> D         svn-test-work\working_copies\merge_tests-132\branch\B\E
>> D         svn-test-work\working_copies\merge_tests-132\branch\D\G\pi
>> D         svn-test-work\working_copies\merge_tests-132\branch\D\H\chi
>> D         svn-test-work\working_copies\merge_tests-132\branch\D\H\omega
>> D         svn-test-work\working_copies\merge_tests-132\branch\D\H\psi
>> D         svn-test-work\working_copies\merge_tests-132\branch\D\H
>>
>> EXCEPTION: SVNLineUnequal
>>
>>
>> There are a few '/' directory separators in the expected output.
>>
>>
>>       Bert
>>
>> ------------------------------------------------------
>> http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2364995
>
> ------------------------------------------------------
> http://subversion.tigris.org/ds/viewMessage.do?dsForumId=495&dsMessageId=2365297

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2365318