subversion-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daniel Shahaf <danie...@elego.de>
Subject Re: svn commit: r1498394 - in /subversion/branches/verify-keep-going/subversion: libsvn_repos/dump.c svnadmin/svnadmin.c
Date Mon, 01 Jul 2013 12:05:52 GMT
prabhugs@apache.org wrote on Mon, Jul 01, 2013 at 12:01:38 -0000:
>    /* Show the summary. */
> -  if (notify_func && keep_going)
> +  if (notify_func && keep_going && found_corruption)
>      {
>        notify_verification_summary(err, notify_func, notify_baton, iterpool);

Thanks.


> +++ subversion/branches/verify-keep-going/subversion/svnadmin/svnadmin.c Mon Jul  1 12:01:38
2013
> @@ -850,13 +850,16 @@ repos_notify_handler(void *baton,
>      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);
> +                              _("\nRevision %ld \n"),
> +                              notify->revision);
> +      if (notify->err)
> +        svn_handle_error2(notify->err, stderr, FALSE /* non-fatal */,
> +                          "svnadmin: ");

How about using "svnadmin: r%ld: " as the prefix, for clarity?

Daniel

Mime
View raw message