[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Printing error stacks.

From: Ivan Zhakov <ivan_at_visualsvn.com>
Date: Sat, 13 Apr 2013 00:17:16 +0400

On Sat, Apr 13, 2013 at 12:14 AM, Branko Čibej <brane_at_wandisco.com> wrote:
> On 12.04.2013 22:06, Ben Reser wrote:
>> On Fri, Apr 12, 2013 at 11:53 AM, Daniel Shahaf <danielsh_at_apache.org> 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
Received on 2013-04-12 22:18:07 CEST

This is an archived mail posted to the Subversion Dev mailing list.