git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] t8005: Nobody writes Russian in shift_jis

From
Brandon Casey <casey@nrlssc.navy.mil>
Date
Jun 19, 2009, 14:54 UTC
Message-ID
<dfYgk9RFOucTCHxtLQsMXejAeKlGJg-R15sTW_RFemcrjsjqoYD0eg@cipher.nrlssc.navy.mil>
In-Reply-To
<7vmy85m0ea.fsf@alter.siamese.dyndns.org>
Junio C Hamano wrote:
Show 24 quoted lines
> The second and third tests of this script expected that Russian strings
> are converted between ISO-8859-5 and Shift_JIS in the "blame --porcelain"
> format output correctly.
> 
> Sure, many platforms may convert between such a combination, but that is
> only because one of the base character set of Shift_JIS, JIS X 0208,
> defines codepoints for Russian characters (among others); I do not think
> anybody uses Shift_JIS when seriously writing Russian, and it is perfectly
> understandable if iconv() libraries on some platforms fail converting
> between this combination, as it does not matter in reality.
> 
> This patch changes the test to verify Japanese strings are converted
> correctly between EUC-JP and Shift_JIS in the same procedure.  The point
> of the test is not about verifying the platform's iconv() library, but to
> see if "git blame" makes correct iconv() library calls when it should.
> 
> We could instead use ISO-8859-5 and KOI8-R as the combination, because
> they are both meant to represent Russian, in order to make this test
> meaningful on more platforms, but we already use Shift_JIS vs EUC-JP
> combinations to test other programs in our test suite, so this combination
> is safer from the point of view of the portability.  Besides, I do not
> read nor write Russian; sorry ;-)
> 
> This change allows tests to pass on my (friend's) Solaris 5.11 box.

No change on my systems. I can convert eucJP and SJIS from/to UTF-8, but I cannot convert between eucJP and SJIS. So tests 2 and 3 still fail for me. Nothing was broken though. The fourth test still passes which converts each of the encodings to UTF-8. So this patch is fine with me.

-brandon
Previous: Alexander GavrilovNext: Junio C Hamano
Message 3 of 4 in “t8005: Nobody writes Russian in shift_jis”
  1. t8005: Nobody writes Russian in shift_jisJunio C Hamano, Jun 19, 2009
  2. Alexander GavrilovJun 19, 2009
  3. Brandon CaseyJun 19, 2009
  4. Junio C HamanoJun 21, 2009

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.