On Mon, Aug 5, 2013 at 7:08 PM, Philip Martin
<philip.mar...@wandisco.com> wrote:
> Lieven Govaerts <l...@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

Reply via email to