On Sun, Oct 17, 2010 at 04:30:41PM +0200, Daniel Shahaf wrote:
> Stefan Sperling wrote on Sun, Oct 17, 2010 at 14:43:57 +0200:
> > Your patch seems to handle copies only. What about locally added files?
>
> Does this part of the regression patch cover the scenario you have in mind?
Yup. It does cover it.
Does this test already pass?
> Index: subversion/tests/cmdline/lock_tests.py
> ===================================================================
> --- subversion/tests/cmdline/lock_tests.py (revision 1023400)
> +++ subversion/tests/cmdline/lock_tests.py (working copy)
> @@ -1573,6 +1590,39 @@ def replace_and_propset_locked_path(sbox):
> +def cp_isnt_ro(sbox):
> ...
> + # added file
> + sbox.simple_add(kappa_path)
> + svntest.actions.set_prop('svn:needs-lock', 'yes', kappa_path)
> + is_writable(kappa_path)
> + sbox.simple_commit(kappa_path)
> + is_readonly(kappa_path)
> ...
> @@ -1619,6 +1669,7 @@ test_list = [ None,
> + cp_isnt_ro,
> ]
