Re: svn commit: rev 2181 - trunk/subversion/tests/clients/cmdline trunk/subversion/tests/clients/cmdline/svntest

Greg Stein <gstein@lyra.org> writes:

> Summary for comments:
>
> * forward-slash seems to work fine on Windows

I believe so. Years ago I used VAX VMS which used '.' for '/' and
surrounded the directory with brackets, the details are vague and I
no longer care ;-)

> * does the .os_path() seem to make sense?

Yup.

> * should .os_path go on the Sandbox? Or on a State? Both? Maybe just a

Sandbox would good except that some of the tests duplicate the working
copy. The Sandbox is anchored in the original working copy so it
won't be able to provide paths into the duplicate. So we need State
as well as, or instead of, Sandbox.

> function in svntest.main?

'sbox.os_path' is shorter than 'svntest.main.os_path' :-)

-- 
Philip
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org