[Bug 52648] New: conhost.exe:tty test_read_console_control() fails in Hindi and Portuguese
https://bugs.winehq.org/show_bug.cgi?id=52648 Bug ID: 52648 Summary: conhost.exe:tty test_read_console_control() fails in Hindi and Portuguese Product: Wine Version: unspecified Hardware: x86-64 OS: Windows Status: NEW Severity: normal Priority: P2 Component: programs Assignee: wine-bugs(a)winehq.org Reporter: fgouget(a)codeweavers.com conhost.exe:tty test_read_console_control() fails on Windows in Hindi and Portuguese: tty.c:1365: Test failed: keyboard state: got 8, expected 18 https://test.winehq.org/data/patterns.html#conhost.exe:tty This can be reproduced on w10pro64-hi, w10pro64-pt-BR and w10pro64-pt-PT but does not fail in the other locales. This failure started when these tests were added: commit 89a8b32d7a976504ee98ba1a7d08574bc9bc00e6 Author: Eric Pouech <eric.pouech(a)gmail.com> AuthorDate: Tue Mar 1 09:26:08 2022 +0100 conhost/tests: Add tests for ReadConsoleW with control. Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52648 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1| |89a8b32d7a976504ee98ba1a7d0 | |8574bc9bc00e6 Keywords| |regression, source, | |testcase -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52648 Eric Pouech <eric.pouech(a)orange.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |eric.pouech(a)orange.fr --- Comment #1 from Eric Pouech <eric.pouech(a)orange.fr> --- well, keyboard mapping is a real pain ;-) the error is not directly linked to the locale, but rather to the keyboard installed on the VM... NB: don't try to run the conhost:tty tests with a French keyboard I did consider also using MapVirtualKeyEx (which would be real clean fix, ie not having the tests being dependant on the installed keyboard). but this API is only half implemented (and the half we'd need is the one missing <g>) I'll loosen up the test -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52648 Eric Pouech <eric.pouech(a)orange.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution|--- |FIXED Fixed by SHA1| |c19f3b620acda9125cc975c0299 | |2a544b27e5929 --- Comment #2 from Eric Pouech <eric.pouech(a)orange.fr> --- looks like its ok now (tty tests still failing on some other parts though) -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52648 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #3 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 7.20. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
WineHQ Bugzilla