[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [test fail] Re: svn commit: r39093 - in trunk/subversion: libsvn_wc tests/cmdline

From: Hyrum K. Wright <hyrum_at_hyrumwright.org>
Date: Wed, 2 Sep 2009 22:15:54 -0500

On Sep 2, 2009, at 8:18 PM, Neels Janosch Hofmeyr wrote:

> Stephen Butler wrote:
>> Author: sbutler
>> Date: Wed Sep 2 16:35:11 2009
>> New Revision: 39093
>>
>> Log:
>> Fix WC-NG handling of "orphaned" items in a replaced dir.
>>
>> "Orphans" are items deleted by a merge that are within a replaced
>> tree, but aren't themselve replaced by the incoming copy operation.
>> They should remain deleted, but were being corrupted.
>>
>> This fix also eliminates the nonsensical "deleted with history"
>> status
>> that used to be displayed for orphaned items.
>>
>> * subversion/libsvn_wc/entries.c
>> (write_entry): Fix the logic for translating schedule-delete entries
>> to wc_db rows.
>>
>> * subversion/libsvn_wc/adm_ops.c
>> (mark_tree): Don't set the "copied" flag for orphans.
>> (process_committed_leaf): Relax an assertion that assumes the copied
>> flag is set for orphans.
>>
>> * subversion/tests/cmdline/copy_tests.py
>> (wc_to_wc_copy_deleted):
>> * subversion/tests/cmdline/revert_tests.py
>> (status_of_missing_dir_after_revert_replaced_with_history_dir):
>> Tweak status expectations for orphaned items: show the wc-rev,
>> don't show the '+' ("with history").
>>
>> * subversion/tests/cmdline/merge_tests.py
>> (copy_then_replace_via_merge): Tweak status expectations. Add
>> update and commit to check for WC corruption.
>> (test_list): Remove XFail() from copy_then_replace_via_merge().
>
> Hi Steve,
>
> hm, this test fails on my side!
> I did a clean checkout & build and got this:
>
> [[[
> $ ./merge_tests.py 134
> subversion/svn/commit-cmd.c:142: (apr_err=160020)
> subversion/libsvn_client/commit.c:853: (apr_err=160020)
> svn: Commit failed (details follow):
> subversion/libsvn_client/commit_util.c:1673: (apr_err=160020)
> subversion/libsvn_delta/path_driver.c:251: (apr_err=160020)
> subversion/libsvn_client/commit_util.c:1469: (apr_err=160020)
> subversion/libsvn_repos/commit.c:364: (apr_err=160020)
> subversion/libsvn_fs_fs/tree.c:1871: (apr_err=160020)
> svn: File already exists: filesystem
> '/home/neels/svn/patch/subversion/tests/cmdline/svn-test-work/
> repositories/merge_tests-134/db',
> transaction '6-6', path '/branch/J/M'
> Traceback (most recent call last):
> File "/home/neels/svn/patch/subversion/tests/cmdline/svntest/
> main.py",
> line 1166, in run
> rc = self.pred.run(sandbox)
> File "/home/neels/svn/patch/subversion/tests/cmdline/svntest/
> testcase.py",
> line 129, in run
> return self.func(sandbox)
> File "./merge_tests.py", line 16220, in copy_then_replace_via_merge
> None, branch_J)
> File "/home/neels/svn/patch/subversion/tests/cmdline/svntest/
> actions.py",
> line 1205, in run_and_verify_commit
> *args)
> File "/home/neels/svn/patch/subversion/tests/cmdline/svntest/
> main.py",
> line 598, in run_svn
> *(_with_auth(_with_config_dir(varargs))))
> File "/home/neels/svn/patch/subversion/tests/cmdline/svntest/
> main.py",
> line 366, in run_command
> None, *varargs)
> File "/home/neels/svn/patch/subversion/tests/cmdline/svntest/
> main.py",
> line 531, in run_command_stdin
> raise Failure
> Failure
> FAIL: merge_tests.py 134: copy then replace via merge
> ]]]
>
> And I don't know what to make of it, really. Hopefully you know?

It's not just you: merge 134 has lit up the buildbots as well.

-Hyrum

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2390508
Received on 2009-09-03 05:16:17 CEST

This is an archived mail posted to the Subversion Dev mailing list.