Re: *sigh*, Windows file open \ vs. / problem

Jay Freeman (saurik) wrote:

>I believe that is what I stated was decided in the first sentence. I
>simply went on to mention a case where it seemed entirely un-intuitive
>:). Then I ended when I realized what the reason probably was (URLs).
>Mainly the reason it felt that way was that "D:/" seemed so weird a
>construction to be passing into an API that thinks in terms of Unix path
>separators. Is there anywhere in the code that assumes that anything
>that doesn't start with '/' is a relative path?
>
I don't think so. We call APR to get an absolute path from whatever's at
hand, and APR generally knows what to do. If not, there's a bug.

>
>
>Sincerely,
>Jay Freeman (saurik)
>saurik@saurik.com
>
>-----Original Message-----
>From: Karl Fogel [mailto:kfogel@newton.ch.collab.net]
>Sent: Monday, February 11, 2002 2:19 PM
>To: Jay Freeman (saurik)
>Cc: dev@subversion.tigris.org
>Subject: Re: *sigh*, Windows file open \ vs. / problem
>
>"Jay Freeman \(saurik\)" <saurik@saurik.com> writes:
>
>>The issue states that there should be a "svn_path_to_internal()"... so
>>the client would be converting it's path from D:\Code\hello to
>>D:/Code/hello? What about on platforms where : is a separator and /
>>
>is
>
>>legal in filenames? (like Mac OS 9, well, the first part, I'm only
>>
>50%
>
>>sure of the second part, and we're just being lucky that / isn't valid
>>on Win32). As far as I'm concerned, the API needs to be accepting the
>>OS's path separators at all points. Oh, wait, I'm seeing it now: the
>>problem with URLs...
>>
>
>I thought we decided that the Subversion APIs would take only
>internal-style paths, and we'd provide an svn_path_to_internal() for
>callers.
>
>>OK, now that I've said that, is that paragraph supposed to be placed
>>
>in
>
>>the issue file instead of to the mailing list? :) (I'm not quite sure
>>about how that is supposed to happen yet.)
>>
>
>Oh, that's always a matter of judgement. In this case, I'd wait for
>more commentary to appear on the list, see if things boil down to a
>consensus, then you can put the consensus in the issue.
>
>-K
>
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
>For additional commands, e-mail: dev-help@subversion.tigris.org
>

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