Re: [PATCH] add 'svnadmin verify' pass to hot-backup.py.in

On 08/19/2010 04:25 PM, Leo Davis wrote:
> Hello,
>
> I've found it useful to verify my backups using 'svnadmin verify' and
> thought it would make a good addition to hot-backup.py.
>
> [[[
> * tools/backup/hot-backup.py.in
> Added command line option "--verify". If flag is present, the
> hotcopy will be verified.
> Added a new pass between the existing steps 3 and 4 (hotcopy and
> compress) that
> invokes 'svnadmin verify' on the hotcopy. If verification fails, the
> script exits with an error.
> ]]]

I like it. Committed in r987379, with some minor tweaks (mostly just
referring to the new backup as a "backup" rather than as a "hotcopy" for
consistency with the rest of the tool's messaging).

Thanks, Leo!

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand