Re: [PATCH] add support for svnrdump to svn-backup-dumps.py

Hello

> 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.
>

Thank you for the comment.

I attach a fixed patch.

[[[
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.__init__) : set flags for --trust-server-cert and
                          --no-auth-cache.
   (SvnBackup.exec_cmd_nt) : open null device and pass it to Popen
                             when printerr is False.
   (SvnBackup.get_extra_param) : get extra option for svn and svnrdump.
                                 always pass --non-interactive to
                                 svn and svnrdump to make sure.
   (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.
   () : add --trust-server-cert.
   () : add --no-auth-cache.
]]]

Regards.

-- 
Masaru Tsuchiyama <m.tmatma_at_gmail.com>