Philip Martin <[email protected]> writes:

> Masaru Tsuchiyama <[email protected]> writes:
>
>>> Is that UTF-8?  It's certainly UTF-8 in the email.
>>
>> No. It is Shift JIS. 
>
> That is the problem.  That string should be UTF-8.  How does it get to
> be  shift-JIS?  The translations should be in UTF-8.  Are you using old
> i18l tools?

Ah! I see.  There are two UTF-8 to native conversions.  It looks like I
did get r1485650 wrong.  At the time I claimed:

> Those two lines are produced by different code paths. The first line
> is produced by repos_notify_handler:
> 
>       svn_error_clear(svn_stream_printf(feedback_stream, scratch_pool,
>                              _("Repository lock acquired.\n"
>                                "Please wait; upgrading the"
>                                " repository may take some time...\n")));
> 
> The second line is produced by:
> 
>   SVN_ERR(svn_cmdline_printf(pool, _("\nUpgrade completed.\n")));
> 
> and svn_cmdline_printf uses svn_cmdline_cstring_from_utf8 to do a UTF8
> to native conversion.
> 
> So it appears the UTF8 to native conversion is missing from
> repos_notify_handler. I think repos_notify_handler should be using
> svn_stream_printf_from_utf8 rather than svn_stream_printf.

(http://svn.haxx.se/dev/archive-2013-05/0416.shtml)

but that seems to be wrong.  I wonder why the user reported the original
problem?

-- 
Philip

Reply via email to