Hyrum K. Wright wrote:
> Lieven Govaerts wrote:
>
>> Hyrum,
>>
>>
>> this is the output of the test on Windows:
>>
>
> Thanks. I'll look at it this weekend. I actually suspect that it's a
> problem with the test.
>
I don't think the test is correct, not only in syntax, it also doesn't
test the feature you've implemented AFAIC.
Attached is a patch for what I think is a good test. What it does is
remove the 'H' directory in r 6, and then checking out H@4 using
externals. Unfortunately this test fails with only the peg rev, unless I
also add the operative rev in the externals definition.
Can you check if it makes sense?
thx,
Lieven
Index: subversion/tests/cmdline/externals_tests.py
===================================================================
--- subversion/tests/cmdline/externals_tests.py (revision 23757)
+++ subversion/tests/cmdline/externals_tests.py (working copy)
@@ -722,7 +722,7 @@
externals_test_setup(sbox)
wc_dir = sbox.wc_dir
-
+
repo_dir = sbox.repo_dir
repo_url = sbox.repo_url
other_repo_url = repo_url + ".other"
@@ -734,10 +734,13 @@
'--password', svntest.main.wc_passwd,
repo_url, wc_dir)
+ # remove A/D/H in the other repo
+ svntest.actions.run_and_verify_svn(None, None, [], 'rm', other_repo_url + '/A/D/H', '-m', 'remove original A/D/H')
+
# Set an external property using peg revision syntax.
new_externals_desc = \
- " -r 1 " + other_repo_url + "/A/D/H@4 exdir_A/H \n" + \
- other_repo_url + "/A/D/G exdir_A/G \n"
+ "exdir_A/H " + other_repo_url + "/A/D/H@4\n" + \
+ "exdir_A/G " + other_repo_url + "/A/D/G\n"
# Set and commit the property.
change_external(os.path.join(wc_dir, "A/D"), new_externals_desc)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Mar 10 00:21:34 2007