Re: Bug Report: Diff fails when using the svn "root" as a path

On 17.11.2014 21:13, Ace Olszowka wrote:
> {{Summary}}
> TortoiseSVN's 'Changed Files' dialog (Diff) fails to show differences
> when showing a difference between an SVN Root for a particular revision.
> The root cause seems to be the desire of the tool to rename the temp
> file with a reserved Windows file name character.
>
> {{Reproducible?}}
> Yes
>
> {{Steps To Reproduce}}
> Attempt to perform a diff between two revisions when the path is the
> "root" address:
>
> TortoiseProc.exe /command:diff /path:"svn://svnserver:1234"
> /startrev:92016 /endrev:92017
>
> Attempting to double click on a file to show the diff will result in an
> error similar to the following:
>
> ---------------------------
> TortoiseSVN
> ---------------------------
> Can't move 'S:\Users\aceo\AppData\Local\Temp\svn-1237D7F9' to
> 'S:\Users\aceo\AppData\Local\Temp\svnserver:1234-rev92016.svn000.tmp': The
> filename, directory name, or volume label syntax is incorrect.
> ---------------------------
> OK
> ---------------------------
>
> {{Work Around}}
> Do not attempt to perform the diff at the root, rather try to perform
> the action at a level below root; IE:
>
> TortoiseProc.exe /command:diff /path:"svn://svnserver:1234/trunk"
> /startrev:92016 /endrev:92017
>
> {{Bug Previously Reported?}}
> Not as far as I can see.
>
> {{Version(s)}}
> Issue appears to be present in the latest stable build of TortoiseSVN
> 1.8.8, Build 25755 - 64bit , 2014/08/10 14:44:06 downloaded from the
> official TortoiseSVN site. This was running on Windows 7 SP1 x64 with
> the latest updates applied.
>
> {{Contact}}
> Reported on the vendors mailing list. If additional information is
> required, it can be provided upon request via the mailing list.
>
> {{Remarks}}
> The bug seems to be related to an attempt to use an invalid character in
> the temporary file path. In this case : is a reserved character as per
> http://msdn.microsoft.com/en-us/library/windows/desktop/aa365247%28v=vs.85%29.aspx#naming_conventions

Thanks for the very detailed report. I've opened issue #674 for this:
https://code.google.com/p/tortoisesvn/issues/detail?id=674

Stefan

-- 
       ___
  oo  // \\      "De Chelonian Mobile"
 (_,\/ \_/ \     TortoiseSVN
   \ \_/_\_/>    The coolest interface to (Sub)version control
   /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=3091533
To unsubscribe from this discussion, e-mail: [users-unsubscribe_at_tortoisesvn.tigris.org].