> -----Original Message-----
> From: phi...@apache.org [mailto:phi...@apache.org]
> Sent: vrijdag 16 maart 2012 10:30
> To: comm...@subversion.apache.org
> Subject: svn commit: r1301390 - in /subversion/trunk/subversion/tests/cmdline:
> svntest/sandbox.py update_tests.py
> 
> Author: philip
> Date: Fri Mar 16 09:29:49 2012
> New Revision: 1301390
> 
> URL: http://svn.apache.org/viewvc?rev=1301390&view=rev
> Log:
> Fallout from the multi-layer-move merge: updates of some moved
> files don't work.
> 
> * subversion/tests/cmdline/svntest/sandbox.py
>   (simple_update): Add optional revision parameter.
> 
> * subversion/tests/cmdline/update_tests.py
>   (update_move_text_mod update_nested_move_text_mod): New XFAIL tests.
>   (test_list): Add new tests.
> 
> Modified:
>     subversion/trunk/subversion/tests/cmdline/svntest/sandbox.py
>     subversion/trunk/subversion/tests/cmdline/update_tests.py
> 
> Modified: subversion/trunk/subversion/tests/cmdline/svntest/sandbox.py
> URL:
> http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/svnte
> st/sandbox.py?rev=1301390&r1=1301389&r2=1301390&view=diff
> =================================================================
> =============
> --- subversion/trunk/subversion/tests/cmdline/svntest/sandbox.py (original)
> +++ subversion/trunk/subversion/tests/cmdline/svntest/sandbox.py Fri Mar 16
> 09:29:49 2012
> @@ -200,14 +200,14 @@ class Sandbox:
>                                    temporary and 'TEMP' or 'PERM',
>                                    parts[1])
> 
> -  def simple_update(self, target=None):
> +  def simple_update(self, target=None, revision='HEAD'):
>      """Update the WC or TARGET.
>         TARGET is a relpath relative to the WC."""
>      if target is None:
>        target = self.wc_dir
>      else:
>        target = self.ospath(target)
> -    svntest.main.run_svn(False, 'update', target)
> +    svntest.main.run_svn(False, 'update', target, '-r', revision)


Note that this change the behavior when target is something like URL@123, as 
svn's -r argument would then default to r123 and not to HEAD.

Maybe you should test if the revision is actually passed instead.

        Bert



Reply via email to