[Box Backup-dev] COMMIT r431 - box/chris/win32/vc2005-compile-fixes/bin/bbackupquery

[Ben Summers]

On 12 Feb 2006, at 11:44, Chris Wilson wrote:

> Hi Ben,
>
>>>  		// Then get it compared
>>>  -		Compare(std::string(DIRECTORY_SEPARATOR) + rLocation,
>>>  +		Compare(std::string("/") + rLocation,
>>>      loc.GetKeyValue("Path"), rParams);
>>
>> The others appear the other way round. What is the intention of  
>> this fix? (in general)
>
> rLocation is a remote path, right? The separator on store paths  
> should ALWAYS be '/', regardless of whatever lame separator our 2- 
> bit platform happens to be using.
>
> Without it, comparing locations on Windows fails, giving up when  
> looking for the location root, becauseit can't find  
> "\ics_data" (for example).

Sounds sensible. So you use / for remote paths, and \ for local paths  
(and for the OS API)? And / everywhere on POSIX?

Ben