Re: svn commit: r1430185 - in /subversion/trunk/subversion: include/svn_dav.h libsvn_ra_serf/options.c libsvn_ra_serf/ra_serf.h mod_dav_svn/version.c

On 01/08/2013 04:20 AM, ivan_at_apache.org wrote:
> Author: ivan
> Date: Tue Jan 8 09:20:54 2013
> New Revision: 1430185
>
> URL: http://svn.apache.org/viewvc?rev=1430185&view=rev
> Log:
> mod_dav_svn: Advertise if server supports sending properties in update
> report in skelta mode.

Ivan,

What motivated this change? You have the server advertising a subfeature of
the REPORT functionality, and the client noting this fact, but the client
isn't actually using the note.

Besides that, it kinda breaks from the pattern we use for REPORT response
functionality, which is that the client says "I want you to respond in X
fashion if possible" and the server, in its response, says either "Sure,
here's my X-style response..." or just "Here's my [implied non-X-style]
response". (See the "add_props_included" report baton member in
libsvn_ra_serf/update.c, and the handling thereof.)

The advertisement you've added seems only valuable if it would prevent a
client from acting *at all* when the server does/doesn't support the inline
properties. Was that your intent?

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Enterprise Cloud Development