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.