Re: Bug + Patch - Command URLs with spaces parsed incorrectly

On 28.05.2012 12:41, Jared Hefty wrote:
> Running the following to invoke the URL handler
>
> tsvncmd:command:showcompare?url1:https://svn/trunk/Scripts/Desert Storm.ini?url2:https://svn/trunk/Scripts/Desert Storm.ini?revision1:229?revision2:230
>
> will report an SVN error, since the command line quoting code will mangle the value portion of url1 and url2 to
>
> url1:"https:"//svn/trunk/Scripts/Desert Storm.ini"
>
>
> Attached is a patch to ensure that the correct portion of the string is quoted.

Thanks for the patch!
Opened issue #318 for this:
http://code.google.com/p/tortoisesvn/issues/detail?id=318

I'll add a comment and a test case to your patch...

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2968029
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].