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

Re: Probable bug with svn copy

From: Noorul Islam K M <noorul_at_collab.net>
Date: Tue, 26 Jan 2010 18:01:26 +0530

Julian Foad <julianfoad_at_btopenworld.com> writes:

> Paul Burba wrote:
>> On Mon, Jan 25, 2010 at 2:17 AM, Noorul Islam K M <noorul_at_collab.net> wrote:
>> > + # Move new added file to another one and commit.
>> > + second_path = os.path.join(new_path, 'second')
>> > + rav_svn(None, None, [], 'move', first_path, second_path)
>> > + rav_svn(None, None, ["Committed revision 4.\n"], 'ci', '-m',
>> ^^^^^^^^^^^^^^^^^^^^^^^^^
>> The expected_stderr is "", not "Committed revision 4.\n". The
>> expected_stdout is something like:
>> [[[
>> Adding svn-test-work\working_copies\copy_tests-77\A\New
>> Adding svn-test-work\working_copies\copy_tests-77\A\New\second
>> Committed revision 4.
>> ]]]
>> So you can either pass None as expected_stdout (in which case it isn't
>> checked at all), pass a single string regex describing the above
>> output, or use svntest.actions.run_and_verify_commit().
> Oops, sorry, I already committed it in r902841. I'll change that or at
> least add a comment.

Thank you, for changing it.

>> Also, would you mind taking a stab at expanding the test to cover
>> Alan's original problem? Specifically that a second working copy,
>> when updated, gets both the move destination *and* the source added.
> Or a separate test - either way would be really helpful.

Please find attached the test case patch for the actual scenario that
Alan explains in his mail.

This patch is created against r899215 because trunk has different
behavior during commit.


New XFail test case for reverse merge, move and update second
working copy scenario. The second working copy update gets both
the move destination and the source added. This issue needs to be
fixed for the test case to pass.

See the email thread

* subversion/tests/cmdline/copy_tests.py
  (reverse_merge_move_update_wc2): New function.
  (test_list): Added test.

Found by: alan.spencer
Suggested by: julianfoad
Patch by: Noorul Islam K M <noorul{_AT_}collab.net>


Thanks and Regards

  • text/x-diff attachment: stored
Received on 2010-01-26 13:32:43 CET

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.