Re: [PATCH] simple optimization and new test case for 'svn cp URL wc'
On Thu, 10 Aug 2006, Madan S. wrote:
> On Wed, 09 Aug 2006 22:55:06 +0530, Daniel Rall <firstname.lastname@example.org> wrote:
> >On Wed, 09 Aug 2006, Madan S. wrote:
> >> Also, I almost broke a case trying one more optimization - Using
> >>svn_wc_adm_open3() instead of svn_wc_adm_probe_open3(). But the tests
> >>passing - of course, that case was not being tested by the
> >>test of copy tests. So, I added it.
> >Could you describe the test case? This is not enough context to
> If a target path of the copy operation already exists and is a directory,
> the source should be copied under the target directory. This specific case
> is not currently tested in repos_to_wc(). I inadverdently broke this
> functionality with some stupid simplifications, but the tests kept
> passing. Hence the itch to fix that.
Thanks Madan! Nice addition. I've committed the copy_tests.py change
to trunk in r21497.
Received on Thu Sep 14 19:04:26 2006
- application/pgp-signature attachment: stored
This is an archived mail posted to the Subversion Dev