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

Chris Wilson boxbackup-dev@fluffy.co.uk
Sun, 12 Feb 2006 11:44:37 +0000 (GMT)


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

Cheers, Chris.
-- 
_ ___ __     _
  / __/ / ,__(_)_  | Chris Wilson <0000 at qwirx.com> - Cambs UK |
/ (_/ ,\/ _/ /_ \ | Security/C/C++/Java/Perl/SQL/HTML Developer |
\ _/_/_/_//_/___/ | We are GNU-free your mind-and your software |