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

Greg Stein wrote:

>On Thu, Jun 13, 2002 at 01:11:31PM +0100, Philip Martin wrote:
>
>
>>Greg Stein <gstein@lyra.org> writes:
>>...
>>
>>
>>>But it is a start... comments?
>>>
>>>
>>One jumps out.
>>
>>The state objects operate with "Subversion paths" like 'A/D/G/rho' but
>>"OS paths" like os.path_join(wc_dir, 'A', 'D', 'G', 'rho') are
>>required for svntest.actions and for working copy manipulations. Your
>>
>>
>
>Well... "required" isn't strictly true :-)
>
>When I was converting the code, I noticed the mixing of path styles. In
>particular, look at line 79 of actions.py (thanks also to Ben for pointing
>out the specific case). We took the '/'-separated paths of main.greek_tree
>and combined them with directories.
>
>And it all seems to work... My assumption is that Windows is allowing
>forward-slashes (which I believe that I've heard people mention is okay).
>
Actually, svn on Windows _requires_ the forward slashes, and Python
_allows_ them. And I made sure the os.path thingies return /-separated
paths by threatening them with Python hacks in main.py.

-- 
Brane Čibej   <brane_at_xbc.nu>   http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org