Daniel Shahaf <danie...@elego.de> writes:

> Philip Martin wrote on Tue, Apr 16, 2013 at 12:40:06 +0100:
>>   $ svn -mm mkdir http://localhost:8888/obj/repo/A
>>   svn: E000013: Can't move 
>> '/home/pm/sw/subversion/obj/repo/db/txn-protorevs/2-b.rev' to 
>> '/home/pm/sw/subversion/obj/repo/db/revs/0/3': Permission denied
>
> Are we sending "13" as an error code across the wire?  That's a system
> error code (EACCES), not an svn error code, so in principle there is no
> guarantee the client can interpret it.  (Not sure if that's a problem in
> practice.)  Shouldn't we be returning an SVN_ERR_* error code?

Yes, we send 13.

HTTP/1.1 500 Internal Server Error\r
Date: Tue, 16 Apr 2013 13:17:44 GMT\r
Server: Apache/2.2.22 (Debian) mod_ssl/2.2.22 OpenSSL/1.0.1e DAV/2 
SVN/1.9.0-dev\r
Content-Length: 291\r
Connection: close\r
Content-Type: text/xml; charset="utf-8"\r
\r
<?xml version="1.0" encoding="utf-8"?>
<D:error xmlns:D="DAV:" xmlns:m="http://apache.org/dav/xmlns"; xmlns:C="svn:">
<C:error/>
<m:human-readable errcode="13">
Can't create directory 
'/home/pm/sw/subversion/obj/repo/db/transactions/2-g.txn': Permission denied
</m:human-readable>
</D:error>

I suppose that's another reason for not returning the low-level errors.


-- 
Certified & Supported Apache Subversion Downloads:
http://www.wandisco.com/subversion/download

Reply via email to