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
signature.asc
Description: OpenPGP digital signature