On Sat, Aug 3, 2013 at 5:36 PM, Masaru Tsuchiyama <m.tma...@gmail.com> wrote: > Hello > > I attach a patch to add support for svnrdump to svn-backup-dumps.py. > > [[[ > add support for svnrdump to svn-backup-dumps.py > > * tools/server-side/svn-backup-dumps.py > () : fix comment > () : update to ver 0.7 > () : import urlparse > (SvnBackup.__init__) : fix message in SvnBackupException constructor. > (SvnBackup.__init__) : check whether the second parameter is a local > path or an URL. > (SvnBackup.__init__) : check whether --deltas option is specified when > the second parameter is an URL. > (SvnBackup.__init__) : set svn and svnrdump paths. > (SvnBackup.exec_cmd_nt) : open null device and pass it to Popen > when printerr is False. > (SvnBackup.get_head_rev) : rename this to get_head_rev_for_local. > (SvnBackup.get_head_rev_for_url) : get HEAD revision for an URL. > (SvnBackup.get_head_rev) : call get_head_rev_for_local for a local > path, get_head_rev_for_url for an URL > (SvnBackup.create_dump) : invoke svnadmin for a local path, > svnrdump for an URL. > () : fix help message. > () : add --svnrdump-path and --svn-path. > ]]]
I guess you don't need --non-interactive because standard input won't be a terminal. But you need a way to set the --username/--password args and maybe --trust-server-cert options to svnrdump. Otherwise looks ok to me.