C. Michael Pilato wrote on Fri, Apr 12, 2013 at 15:16:23 -0400:
> On 04/12/2013 02:53 PM, Daniel Shahaf wrote:
> > Another idea: avoid printing the (apr_err=) part if it's the same code as the
> > line above it (which should be the common case):
> >
> > subversion/svn/checkout-cmd.c:168 (apr_err=SVN_ERR_RA_LOCAL_REPOS_OPEN_FAILED)
> > subversion/libsvn_client/checkout.c:197 (apr_err=^)
> > subversion/libsvn_client/checkout.c:100 (apr_err=^)
> > subversion/libsvn_client/ra.c:541 (apr_err=^)
> > subversion/libsvn_client/ra.c:393 (apr_err=^)
> > subversion/libsvn_ra/ra_loader.c:482 (apr_err=^)
> > svn: E180001: Unable to connect to a repository at URL 'file:///home/cmpilato/tests/arch'
> > subversion/libsvn_ra_local/ra_plugin.c:578 (apr_err=^)
> > svn: E180001: Unable to open an ra_local session to URL
> > subversion/libsvn_ra_local/split_url.c:46 (apr_err=SVN_ERR_SOMETHING_ELSE)
> > svn: E180003: Unable to open repository 'file:///home/cmpilato/tests/arch'
>
> I like the brevity. One concern that I have with this (and with the current
> output, for that matter) is that there is really nothing that tells me that
> E180003 means SVN_ERR_RA_LOCAL_REPOS_OPEN_FAILED. I've got a line which
> lists the symbolic name, a bunch of lines with effectively say "ditto", and
> then a line of actual "meat" that refers only to a number and *not* the name.
>
The last two lines are something like:
subversion/libsvn_ra_local/split_url.c:46 (apr_err=SVN_ERR_SOMETHING_ELSE)
subversion/libsvn_ra_local/foobar.c:146 (apr_err=^)
svn: E180003: Unable to open repository 'file:///home/cmpilato/tests/arch'
The symbolic name of E180003 is SVN_ERR_SOMETHING_ELSE, you get to it by
looking for the closest symbolic name above E180003.
Received on 2013-04-12 21:22:29 CEST