Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WSL fails to return tty screen size given correct VT sequence #4327

Closed
therealkenc opened this issue Jul 21, 2019 · 4 comments
Closed

WSL fails to return tty screen size given correct VT sequence #4327

therealkenc opened this issue Jul 21, 2019 · 4 comments

Comments

@therealkenc
Copy link
Collaborator

therealkenc commented Jul 21, 2019

[regrouping]

@DHowett-MSFT
Copy link

Just a note: I'm having trouble with this repro.

echo -e '\e[18t' > $(tty) | xxd redirects the output directly to the tty, rendering the additional redirect into xxd useless.

echo -e '\e[18t' | xxd redirects the output directly into xxd, which is why xxd then prints the raw bytes 1B 5B 31 38 74. This is not what you're expecting (it dumps the value of CSI 18 t instead of the response CSI 8).

Regardless, that's not the correct way to reproduce this. You want echo -en '\e[18t'; xxd, at which point the terminal emulator will respond with CSI 8 ; <h> ; <w> t. You'll have to press Ctrl+D to get xxd to recognize the end of the stream, as there's no delineation in the input stream between terminal reports and user input.

That just turns this into a feature request that we support CSI 18 t. I'll rename and categorize this bug appropriately.

@DHowett-MSFT
Copy link

Oh, I can't, as I forgot this wasn't the terminal repo. This is just a feature request for conhost.

@therealkenc
Copy link
Collaborator Author

Sh*t sorry Dustin. That was my bad cut and paste. I'll delete both in two min and regroup.

@therealkenc
Copy link
Collaborator Author

therealkenc commented Jul 22, 2019

Don't have the power to delete; closed is the best I can do. I'd have to take harder run at a wsl.exe-as-child (eg rawpty.cpp) and post if (iff only if) I can demonstrate a gap. This is more challenging than the half hour exercise I was hoping for last weekend because wsl.exe is a black box and ssh.exe is not using ConPTY. Which is not on you. If it can be done in-scope xterm-256color wsl.exe scenario by whatever means then this is by-design ConPTY, and feature request wsl.exe -t "Force pseudo-terminal allocation" option WSL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants