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

Re: svn up --depth empty breaks working copy

From: C. Michael Pilato <cmpilato_at_red-bean.com>
Date: Thu, 29 May 2008 08:52:07 -0400

David Glasser wrote:
> On Wed, May 28, 2008 at 11:42 PM, Stefan Küng <tortoisesvn_at_gmail.com> wrote:
>> David Glasser wrote:
>>> I can reproduce this with 1.4.x svnserve but not trunk svnserve.
>>>
>>> Note that the actual over-the-wire communication seems identical,
>>> which is kind of wacky. But maybe the lack of depth capability is
>>> triggering some compat code, which is broken?
>> Finally got the svn.exe built here.
>> I can reproduce the problem from here too (on XP SP3) with svn.exe built
>> from the 1.5.x branch.
>>
>> Since you already got a reproduction recipe, I'm not trying to create
>> another one. But just as an additional info: my repository is on apache, not
>> svnserve (svn version 1.4.6 on the apache server). So it's nothing to do
>> with svnserve.
>
> The same problem code is in all three remote ra layers. This fixes
> the issue; currently running tests to see if it does any harm.
>
> cmpilato, any insight into why you wrote the code this way originally?

Sorry, I have no memory of writing this code at this point. (I mean,
besides knowing that I was the one who wrote it.) There's some chance it
was a botched logic reduction (that code might have at some point been "if
dir and some condition or if file and some other condition").

Anyway, the tweak to okay_to_edit() looks correct.

> (Also, why libsvn_ra_svn/client.c(ra_svn_get_reporter) uses
> svn_error_clear in its call to svn_delta_depth_filter_editor?)

Ooh. I probably deferred the change to ra_svn_get_reporter() to return an
error (and its callers to expect it) because I was just trying to get
something coded up, and failed to go back and correct that before committing up.

-- 
C. Michael Pilato <cmpilato@red-bean.com> | http://cmpilato.blogspot.com/
"The Christian ideal has not been tried and found wanting.  It has
  been found difficult; and left untried."  -- G. K. Chesterton
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-05-29 14:52:21 CEST

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

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