On Sat, Apr 13, 2013 at 12:14 AM, Branko Čibej <[email protected]> wrote: > On 12.04.2013 22:06, Ben Reser wrote: >> On Fri, Apr 12, 2013 at 11:53 AM, Daniel Shahaf <[email protected]> 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 this idea a lot. > > I somwhat like it, too, but I'd simply remove the "(apr_err=...)" in the > lines that just refer to a previoius symbolic name. > +1.
-- Ivan Zhakov
