On Fri, Jun 28, 2013 at 01:11:18PM +0000, Daniel Shahaf wrote:
> On Fri, Jun 28, 2013 at 12:56:27PM -0000, prabhugs_at_apache.org wrote:
> > @@ -1532,6 +1610,18 @@ svn_repos_verify_fs3(svn_repos_t *repos,
> > + {
> > + struct error_list *err_list = APR_ARRAY_IDX(error_summary, i, struct error_list *);
> > + notify_verification_error_summary(err_list->rev, err_list->err, notify_func, notify_baton,
>
> Wrap to 80 columns.
>
> > Modified: subversion/branches/verify-keep-going/subversion/svnadmin/svnadmin.c
> > URL: http://svn.apache.org/viewvc/subversion/branches/verify-keep-going/subversion/svnadmin/svnadmin.c?rev=1497757&r1=1497756&r2=1497757&view=diff
> > ==============================================================================
> > --- subversion/branches/verify-keep-going/subversion/svnadmin/svnadmin.c (original)
> > +++ subversion/branches/verify-keep-going/subversion/svnadmin/svnadmin.c Fri Jun 28 12:56:26 2013
> > @@ -847,6 +847,18 @@ repos_notify_handler(void *baton,
> > "svnadmin: ");
> > return;
> >
> > + case svn_repos_notify_failure_summary:
> > + if (notify->revision != SVN_INVALID_REVNUM)
> > + cmdline_stream_printf(feedback_stream, scratch_pool,
> > + _("r%ld: %s\n"),
> > + notify->revision, notify->err->message);
>
> err->message may be NULL, perhaps svn_err_best_message()?
Should be using svn_error_root_cause() as well, otherwise output
in maintainer-mode builds looks like this:
-----Summary of corrupt revisions-----
r5: traced call
r6: traced call
r7: traced call
r8: traced call
Received on 2013-06-28 15:46:56 CEST