Yay, a bikeshed discussion!

[Daniel Shahaf]
>     % $svn upgrade /
>     subversion/svn/upgrade-cmd.c:73: (apr_err=_)
>     subversion/libsvn_client/upgrade.c:112: (apr_err=_)
>     subversion/libsvn_wc/upgrade.c:2194: (apr_err=_)
>     subversion/libsvn_wc/wc_db.c:15006: (apr_err=SVN_ERR_WC_INVALID_OP_ON_CWD)
>     svn: E155019: Can't upgrade '/' as it is not a working copy root
>     subversion/libsvn_wc/wc_db_util.c:134: (apr_err=_)
>     svn: E000002: Working copy database '/.svn/wc.db' not found
>     subversion/libsvn_wc/wc_db.c:15006: (apr_err=_)
>     svn: E000002: Additional errors:
>     subversion/libsvn_wc/old-and-busted.c:1201: (apr_err=_)
>     subversion/libsvn_wc/adm_files.c:294: (apr_err=_)
>     subversion/libsvn_subr/stream.c:850: (apr_err=_)
>     subversion/libsvn_subr/io.c:3271: (apr_err=2)
>     svn: E000002: Can't open file '/.svn/entries': No such file or directory
>     zsh: exit 1     $svn upgrade /

What is the utility of the "(apr_err=_)" suffix there?  I think it's
better to omit it.  Print apr_err= only when it is new information.

Reply via email to