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

Windows terminal issues caused by winpty #45693

Closed
Tyriar opened this issue Mar 13, 2018 · 21 comments · Fixed by #65569
Closed

Windows terminal issues caused by winpty #45693

Tyriar opened this issue Mar 13, 2018 · 21 comments · Fixed by #65569
Assignees
Labels
plan-item VS Code - planned item for upcoming terminal General terminal issues that don't fall under another label windows VS Code on Windows issues

Comments

@Tyriar
Copy link
Member

Tyriar commented Mar 13, 2018

This is a meta issue to act as a central place for all winpty issues.

Legend

Icon Meaning
✔️ Workaround in place
✖️ Still an issue

General

Foreground/background color

Process killing / ctrl+c

Clearing

Tasks

Windows regressions

These issues slipped into stable Windows and took month(s) to get a fix to users as a result.

@Tyriar Tyriar added plan-item VS Code - planned item for upcoming terminal General terminal issues that don't fall under another label windows VS Code on Windows issues labels Mar 13, 2018
@Tyriar Tyriar added this to the Backlog milestone Mar 13, 2018
@Tyriar Tyriar self-assigned this Mar 13, 2018
@monk-time
Copy link

Shouldn't #45682 be also added to this list?

@Tyriar
Copy link
Member Author

Tyriar commented Mar 16, 2018

@monk-time added it to the list

@jarbus
Copy link

jarbus commented Nov 14, 2018

Not sure if this is of any help, but when opening the bash terminal it additionally scrolls all the way down after highlighting the entire terminal. Killing it and opening a new terminal just goes back to the regular highlighting issue. The following image is right after opening a terminal just after opening VSCode
image
This is additionally on a machine that got a fresh installation of windows an hour prior.

@Astrantia
Copy link

Astrantia commented Nov 18, 2018

@Tyriar It didn't make into 1.29. Are you planning to release the improvements in 1.30?

@ghiscoding this year, I'll be working on this again next month so it will hopefully make it into 1.29.

@cedric05
Copy link
Contributor

with latest fall creators update, windows is psesdo console apis. is vscode planning to use them?
Reference: https://docs.microsoft.com/en-us/windows/console/creating-a-pseudoconsole-session
@Tyriar
i want to give it a try

@ExE-Boss
Copy link
Contributor

@cedric05 This is being done in microsoft/node-pty#236 (see microsoft/node-pty#216 for details).

@namvt13
Copy link

namvt13 commented Nov 25, 2018

I was redirected here by another user, but the issue at #26375 & #47088 has also happened outside of vscode (In my case, it is the terminal of Ubuntu), so it likely to be some OS bug.

@Tyriar
Copy link
Member Author

Tyriar commented Nov 26, 2018

Adoption of conpty is on the way as @ExE-Boss pointed out, it's currently blocked on VS Code adopting Electron 3 #52629

@Astrantia
Copy link

@Tyriar is this shipped in exploration builds if electron 3 is the blocker?

@SetTrend
Copy link

cls still doesn't clear scroll buffer on Windows systems. Rather annoying. Particularly when you need to search through long logs.

https://docs.microsoft.com/en-us/windows-server/administration/windows-commands/cls
https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.core/functions/clear-host

@Tyriar
Copy link
Member Author

Tyriar commented Dec 21, 2018

ConPTY support is landing in the next Insiders build, this uses a new backend for the terminal on WIndows 10 build number 17692+, this is expected to fix many bugs in the terminal on Windows as we're adopting Microsoft's official pty API. For earlier versions winpty will still be used a many bugs will still remain (unless they're fixed in that project).

If you find any issues with conpty please report them to https://github.com/Microsoft/console (the team who own ConPTY) or this repo (if you think it's an integration issue). You can opt out of conpty by adding this setting:

"terminal.integrated.windowsEnableConpty": false

@microsoft microsoft locked as resolved and limited conversation to collaborators Dec 21, 2018
@Tyriar
Copy link
Member Author

Tyriar commented Jan 23, 2019

Update: Windows build number 18309+ is now required for ConPTY to be turned on as it was not stable on 17692.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
plan-item VS Code - planned item for upcoming terminal General terminal issues that don't fall under another label windows VS Code on Windows issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.