libs/newlib-cygwin
libs
/
newlib-cygwin
mirror of git://sourceware.org/git/newlib-cygwin.git
4
0
Fork 0
Code Issues Projects Releases Wiki Activity

Cygwin: pty: Prevent garbled output for existing non-cygwin apps. 

- If pseudo console is disabled, non-cygwin apps do not detect
  console device. In this case, some apps output UTF-8 regardless
  of the locale setting. At least git-for-windows, rust-based apps
  and node.js do that. This patch provides backward compatibility
  as default behaviour by setting console codepage to the charset of
  the locale. Even in the cases above, garbled output is prevented
  with this patch in most cases because mintty uses UTF-8 by default.

  I beleave this is not really a problem in cygwin side but that in
  app side, however, some users complain about garbled output with
  existing apps in MSYS2 (which is based on cygwin) in which pseudo
  console is disabled by default.
This commit is contained in:
Takashi Yano via Cygwin-patches 2020-09-11 19:54:40 +09:00 committed by Corinna Vinschen
parent 09738c3062
commit 232fde0e76
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
winsup/cygwin/fhandler_tty.cc
View File

@ -1830,7 +1830,11 @@ fhandler_pty_slave::setup_locale (void)
extern UINT __eval_codepage_from_internal_charset ();
if (!get_ttyp ()->term_code_page)
get_ttyp ()->term_code_page = __eval_codepage_from_internal_charset ();
{
get_ttyp ()->term_code_page = __eval_codepage_from_internal_charset ();
SetConsoleCP (get_ttyp ()->term_code_page);
SetConsoleOutputCP (get_ttyp ()->term_code_page);
}
}
void