Re: [PATCH] Implement svnadmin verify --force

Bert Huijben wrote:

>> +1 to having a non-zero exit code if there was any error throughout.
>
> In that case: why do we add --force?

The option is called --keep-going.

> Maybe we should default to this new behavior and *add* a quick exit on error
> for whoever needs it.
>
> I think the total report on which revisions are broken is more informational
> than just the first error. And I don't like the '--force' for
> argument for
> continuing.

Near the start of this thread we mentioned the main use cases.  There are basically two:

* Confirm that the repository is in a good state (e.g. before/after a backup);

* Diagnostic mode for use when a problem has been encountered.

Keep-going mode is more useful for the latter; stop-quickly mode *may* be more useful (and is certainly fine) for the former.  So it's not obvious that changing the default mode is a good idea.

- Julian
Received on 2012-11-01 18:26:50 CET