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. -- C. Michael Pilato <cmpil...@collab.net> CollabNet <> www.collab.net <> Enterprise Cloud Development
signature.asc
Description: OpenPGP digital signature
