On 02/04/2013 09:55 AM, C. Michael Pilato wrote:
> On 02/04/2013 06:30 AM, Philip Martin wrote:
>> jinfroster <jinfros...@mail.ru> writes:
>>
>>> Add "SVNKeywordSubstitution" per-directory (repository) mod_dav_svn
>>> configuration parameter (default is "Off"). Implement keywords
>>> substitution.
>>
>>> * subversion/mod_dav_svn/repos.c
>>>   (set_headers):
>>>     If parameter SVNKeywordSubstitution is On, don't send
>>>     "Content-length". We can't guess the size of expanded stream at
>>>     the moment (..is that bad?)
>>>   (deliver):
>>>     If parameter SVNKeywordSubstitution is On, perform keywords
>>>     substitution, like client-side utilities do.
>>
>> Does your Subversion client use neon?  I think this causes the server to
>> send expanded keywords in response to GET requests and so will break
>> Subversion clients using serf.
> 
> Yes, that's precisely what it does.  But the problem isn't limited to Serf
> clients.  Any call to svn_ra_get_file() -- for example, to support 'svn cat'
> -- will suffer.  So, yeah, cool idea, but unfortunately the patch is
> unacceptable as-is.

Sorry, hit send too fast.  That should be "... Any call to svn_ra_get_file()
with Neon ..."


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

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to