On Mon, Aug 5, 2013 at 7:08 PM, Philip Martin
<philip.martin_at_wandisco.com> wrote:
> Lieven Govaerts <lgo_at_apache.org> writes:
>
>> Can you test if attached patch fixes this issue?
>
>> Index: subversion/libsvn_ra_serf/util.c
>> ===================================================================
>> --- subversion/libsvn_ra_serf/util.c (revision 1510435)
>> +++ subversion/libsvn_ra_serf/util.c (working copy)
>> @@ -2434,6 +2434,10 @@ svn_ra_serf__error_on_status(serf_status_line slin
>> "server or an intermediate proxy does not accept "
>> "chunked encoding. Try setting 'http-chunked-requests' "
>> "to 'auto' or 'no' in your client configuration."));
>> + case 501:
>> + return svn_error_createf(SVN_ERR_UNSUPPORTED_FEATURE, NULL,
>> + _("The requested feature is not supported by "
>> + "'%s'"), path);
>> }
>>
>> if (sline.code >= 300)
>
> Yes, it does. It also affects mergeinfo:
>
> $ svn1.8 mergeinfo ^/ wc
> svn: E175002: Unexpected HTTP status 501 'Method Not Implemented' on '/obj/repo/!svn/bc/1/A'
> svn: E200007: Additional errors:
> svn: E200007: The requested report is unknown.
>
> $ subversion/svn/svn mergeinfo ^/ wc
> svn: E200007: Retrieval of mergeinfo unsupported by 'http://localhost:8888/obj/repo/A'
>
To be clear, this mergeinfo error is to be expected, and this patch
has improved the error message. Or did you see a regression here?
Lieven
> --
> Philip Martin | Subversion Committer
> WANdisco | Non-Stop Data
Received on 2013-08-05 21:48:42 CEST