> On Tue, 2005-12-20 at 10:27 -0800, Daniel Rall wrote:
> > How about adding --non-interactive to run_and_verify_svn(),
> > run_svnsync(), etc. (as appropriate)?
> > > # Initialize the mirror repository from the master.
> > > output, errput = svntest.main.run_svnsync(
> > > - "initialize", dest_sbox.repo_url, "--source-url", sbox.repo_url)
> > > + "initialize", dest_sbox.repo_url, "--source-url", sbox.repo_url,
> > > + "--username", svntest.main.wc_author,
> > > + "--password", svntest.main.wc_passwd)
> I've often thought it would be nice to get rid of those extra
> arguments by adding them to the test suite's default config file
> (which is currently empty, I believe). That wouldn't automatically
> simplify tests that create their own config file, but there aren't
> many of those.
I like the sound of that -- which arguments were you thinking I could
refactor into the config file?
On Tue, 20 Dec 2005, Michael Pilato wrote:
> Not all 'svn' subcommands accept the --non-interactive flag (and
> specifying it for one such subcommand results in an error).
Yeah, the subcommands which don't accept --non-interactive include:
Do you think I should add the --non-interactive flag on a case-by-case
basis, rather than manipulating the test code to add it only for
certain subcommands (e.g. by examining the argument list)?
Received on Sun Dec 25 04:29:29 2005
- application/pgp-signature attachment: stored