On 02/04/2013 09:55 AM, C. Michael Pilato wrote:
> On 02/04/2013 06:30 AM, Philip Martin wrote:
>> jinfroster <jinfroster_at_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 <cmpilato_at_collab.net>
CollabNet <> www.collab.net <> Enterprise Cloud Development
Received on 2013-02-04 15:59:13 CET