Re: Printing error stacks. (was: Re: svn_error_symbolic_name)

Stefan Sperling wrote on Sat, Apr 13, 2013 at 10:39:50 +0200:
> On Sat, Apr 13, 2013 at 03:23:32AM +0300, Daniel Shahaf wrote:
> > % $svnadmin create /
> > svnadmin: E200011: Repository creation failed
> > svnadmin: E200011: Could not create top-level directory
> > svnadmin: E200011: '/' exists and is non-empty
> > (E200011 = DIR_NOT_EMPTY)
> > %
> >
> > Daniel
>
> I like it. Can we make trunk use this for now?
>

I am +1 on trunk's current output, in both release mode and maintainer
mode. I don't have a strong opinion on whether the above release-mode
change would be an improvement.

> I think I'd prefer using 1.7-style error reporting in 1.8 until we've
> found a solution everyone is happy with. There is no rush, we can make
> this change for 1.9 just as well.

1.7, 1.8, and trunk_at_HEAD have identical output in release mode.

The latter two print the maintainer-mode stack traces differently, and
in a way that most devs find to be an improvement. I'd rather let it
stay, because it's useful.

There is a change nominated to 1.8.x/STATUS that only affects the
maintainer-mode output, too. I'll go and ensure the STATUS entry
has a clear justification.
Received on 2013-04-13 14:49:28 CEST