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

Re: svn commit: r1143860 - /subversion/trunk/subversion/libsvn_ra_serf/update.c

From: Ivan Zhakov <ivan_at_visualsvn.com>
Date: Thu, 7 Jul 2011 20:45:02 +0400

On Thu, Jul 7, 2011 at 19:17, C. Michael Pilato <cmpilato_at_collab.net> wrote:
> On 07/07/2011 10:55 AM, ivan_at_apache.org wrote:
>> Author: ivan
>> Date: Thu Jul  7 14:55:19 2011
>> New Revision: 1143860
>>
>> URL: http://svn.apache.org/viewvc?rev=1143860&view=rev
>> Log:
>> Fix calculation X-SVN-VR-Base request header in ra_serf.
>>
>> This commit reverts r1142065 and r1143089.
>>
>> * subversion/libsvn_ra_serf/update.c
>>   (report_dir_t): Remove repos_relpath.
>>   (report_context_t): Remove root_is_switched.
>>   (start_report): Do not update repos_relpath.
>>   (end_report): Use only wcprops as source for delta base. Our current
>>    protocol doesn't gives us stable way to construct delta base without
>>    wcprops. We can optimize it later.
>
> Whoa!  wcprops again?  Can you explain this?  wcprops only ever existed
> because it was too expensive to ask the server to give us a URL to describe
> a particular object.  But URL generation is *supposed* to be trivial in
> HTTPv2, so... what's up?!
>
We cannot calculate them even over HTTPv2 in case of switched paths
(or file externals). Bert implemented some heuristic in r1142065, but
as any heuristic method. Really stable way to fix the problem is to
include property with repos relative path to update report.

The problem that old code actually didn't work: X-SVN-VR-Base request
header was simply ignored by server (see r1141845).

-- 
Ivan Zhakov
Received on 2011-07-07 18:45:59 CEST

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