public inbox for cygwin-developers@cygwin.com
 help / color / mirror / Atom feed
From: Thomas Wolff <towo@towo.net>
To: Takashi Yano <takashi.yano@nifty.ne.jp>, cygwin-developers@cygwin.com
Subject: Re: [PATCH v3 0/1] Pseudo console support in PTY (v3)
Date: Sat, 06 Apr 2019 21:33:00 -0000	[thread overview]
Message-ID: <4b4aae13-877c-8521-54c5-dcf2dafbc66d@towo.net> (raw)
In-Reply-To: <20190406111308.1822-1-takashi.yano@nifty.ne.jp>

Hi Takashi,

> The major changes from v2 is as follows.
My testing showed the following observations.
> ...
>
> (3) Code page handling is refined a bit. Now, code page 65001 (UTF8)
>       and native code page in your region should work without garbled
>      characters. This works properly only on terminal which supports
>      UTF-8.
Character output conversion works with my test program (using 
WriteConsoleW). Using java, output works too (with proper java encoding 
option), but non-ASCII input characters are replaced by and echoed as space.

If I try a non-Unicode terminal session, e.g. running mintty -o Locale=C 
-o Charset=CP1252 -, and enter a € key, mintty hangs. While you said it 
doesn't work (yet), it should better not hang at least. I'd like to 
repeat at this point that the whole mechanism should only be applied if 
a non-cygwin program is run. (If it were, entering € in bash would not 
block the terminal.)

> (4) Synchronization between real terminal and pseudo console screen
>      buffer is improved. For this, screen is cleared on startup of pty
>      slave app. Also echo for key input is pushed into pseudo console
>      screen buffer.
Do you actively synchronize anything? I thought this is not needed 
anymore (unlike winpty) because the ConPTY performs the adaptation.

Terminal reports "\033[6n" and "\033[0c" are apparently emulated and 
sent in reverse order;
response to primary device attribute request is wrong (not what mintty 
would report).
There is no response to secondary device attribute request ("\033[>0c") 
and others,
e.g. '\033[18t' or '\033]10;?\033\' and many others; why can't these 
requests be passed to the terminal and handled transparently? (You 
explained something around handles before but I don't get the point.)

Output to alternate screen seems to be forced to the primary screen, mostly;
if I try to switch screen in various ways (menu, echo "\033[?1047h" > 
/dev/pty1, before or while cmd.exe runs),
behaviour appears to be inconsistent and not as expected (expected 
behaviour is that any output goes to the active screen). Again, I don't 
see a need that you intercept this at all.

Thomas

  parent reply	other threads:[~2019-04-06 21:33 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-03-30 13:08 Pseudo console support in PTY Takashi Yano
2019-03-30 19:47 ` Corinna Vinschen
2019-03-30 19:59   ` Corinna Vinschen
2019-03-30 23:07 ` Thomas Wolff
2019-03-31 14:38   ` Corinna Vinschen
2019-03-31 15:00   ` Takashi Yano
2019-04-02 11:02 ` Corinna Vinschen
2019-04-02 17:16   ` Thomas Wolff
2019-04-02 17:51     ` Corinna Vinschen
2019-04-03  7:18       ` Thomas Wolff
2019-04-03  7:28         ` Corinna Vinschen
2019-04-03  7:55           ` Thomas Wolff
2019-04-03  8:02             ` Corinna Vinschen
2019-04-03 11:33               ` Thomas Wolff
2019-04-03 12:17                 ` Corinna Vinschen
2019-04-04  4:17                   ` Takashi Yano
2019-04-04  8:06                     ` Corinna Vinschen
2019-04-04  4:15             ` Takashi Yano
2019-04-03 16:36   ` [PATCH v2 0/1] Pseudo console support in PTY (v2) Takashi Yano
2019-04-03 16:37     ` [PATCH v2 1/1] Cygwin: pty: add pseudo console support Takashi Yano
2019-04-03 16:50     ` [PATCH v2 0/1] Pseudo console support in PTY (v2) Corinna Vinschen
2019-04-04  5:27       ` Takashi Yano
2019-04-04  8:17         ` Thomas Wolff
2019-04-04  9:34           ` Takashi Yano
2019-04-03 17:11     ` Corinna Vinschen
2019-04-04  8:59       ` Takashi Yano
2019-04-04 10:46         ` Corinna Vinschen
2019-04-06 11:13           ` [PATCH v3 0/1] Pseudo console support in PTY (v3) Takashi Yano
2019-04-06 11:14             ` [PATCH v3 1/1] Cygwin: pty: add pseudo console support Takashi Yano
2019-04-06 17:43             ` [PATCH v3 0/1] Pseudo console support in PTY (v3) Corinna Vinschen
2019-04-12 10:22               ` [PATCH v4 0/1] Pseudo console support in PTY (v4) Takashi Yano
2019-04-12 10:23                 ` [PATCH v4 1/1] Cygwin: pty: add pseudo console support Takashi Yano
2019-04-12 12:29                 ` [PATCH v4 0/1] Pseudo console support in PTY (v4) Corinna Vinschen
2019-04-15  8:18                   ` Corinna Vinschen
2019-04-15 23:17                     ` Takashi Yano
2019-04-14 15:23                 ` [PATCH v5 0/1] Pseudo console support in PTY (v5) Takashi Yano
2019-04-14 15:23                   ` [PATCH v5 1/1] Cygwin: pty: add pseudo console support Takashi Yano
2019-04-15  8:38                     ` Corinna Vinschen
2019-04-16  0:41                       ` Takashi Yano
2019-04-16  9:16                         ` Corinna Vinschen
2019-06-24 10:53                           ` Corinna Vinschen
2019-07-25 14:31                             ` Corinna Vinschen
2019-08-08 19:24                               ` Corinna Vinschen
2019-08-12 12:07                             ` Takashi Yano
2019-08-12 12:50                               ` Corinna Vinschen
2019-08-12 14:36                                 ` Takashi Yano
2019-04-14 16:06                   ` [PATCH v5 0/1] Pseudo console support in PTY (v5) Takashi Yano
2019-04-16  1:49                     ` Takashi Yano
2019-04-16  1:51                       ` Takashi Yano
2019-04-06 21:33             ` Thomas Wolff [this message]
2019-04-07  5:05               ` [PATCH v3 0/1] Pseudo console support in PTY (v3) Takashi Yano
2019-04-07 12:02                 ` Takashi Yano
2019-04-07 21:21                 ` Thomas Wolff

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4b4aae13-877c-8521-54c5-dcf2dafbc66d@towo.net \
    --to=towo@towo.net \
    --cc=cygwin-developers@cygwin.com \
    --cc=takashi.yano@nifty.ne.jp \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).