Re: [TSVN] blame won't work

SteveKing wrote:
> Simon Large wrote:
> [snip]
>>> Commit failed (details follow):
>>> MERGE request failed on 'svn/MyTest/%E6%B8%AC%E8%A9%A6'
>>> General svn error from server.
>>>
>>> Is this a bug of Subversion or TSVN?
>>> How can I solve it?
>>
>>
>> Looks like TSVN should be unescaping the subversion error message
>> before displaying it. But we have had other examples recently (also
>> 1.1.7) where the message is displaying correctly. Any ideas anyone?
>
> No, that's not TSVN's fault. Subversion returns the error message.
> TSVN has no way of telling if the error message contains an URL or
> not (and no, looking for URL's won't really work reliably if you
> start thinking about it).

It's not a bug, just difficult to read. What will go wrong if we
unescape all messages? Are there any messages which contain a '%' sign
which is not part of an escaped URL?

> But I don't quite see the problem here: the URL is completely legal!
> That's the real URL used by browsers too.

It is legal, but not legible ;-) We allow people to enter URLs in their
native language and then escape them before using them. So they might
reasonably expect us to unescape them when we give them back. But if
it's not a simple fix then we can just add that to the docs or FAQ.

Simon

-- 
       ___
  oo  // \\      "De Chelonian Mobile"
 (_,\/ \_/ \     TortoiseSVN
   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
   /_/   \_\     http://tortoisesvn.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org