Alexey Shumkin <Alex.Crezoff@xxxxxxxxx> writes: > 4. [PATCH v6 4/5] pretty: Add failing tests: --format output should honor logOutputEncoding > iso8859-5 encoding reverted back to cp1251 encoding (as it was in v4 series) Thanks for a reroll, but why this change? The reason I asked you to avoid 8859-5 is because our test do not use that encoding and I do not want to add new dependency to people when they run test. cp1251 shares exactly the same issue, doesn't it? So in that sense, this change does not make anything better. That is why I asked you if the breakage you are trying to demonstrate about non-ASCII non-UTF8 encoding was specific to Cyrillic/Russian. I do not recall seeing your answer, but what is the right thing to do depend on it. - If the answer is yes, then we would need to add dependency either way, and 8859-5 is just as fine as cp1251. - If the breakage is not specific to Cyrillic, it should be reproducible using 8859-1 (latin-1), and our tests already depend on 8859-1, so we wouldn't be adding new dependencies on people. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html