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

Suggestion: winpty WSL PTY support #148

Closed
lars18th opened this issue Jul 18, 2018 · 2 comments
Closed

Suggestion: winpty WSL PTY support #148

lars18th opened this issue Jul 18, 2018 · 2 comments

Comments

@lars18th
Copy link

Hi @rprichard ,

After reading about inverted-wslbridge.exe at mintty/wsltty#41 (comment) ,
I request why the winpty tool can't implement WSL PTY support (aka "WSL winpty adapter").

I like to use the WSL shell as my regular terminal in Windows, for both Windows and WSL Linux command tools. At time, using the wslbridge I can open the WSL shell in any Terminal (CMD, Cygterm, SSHd, etc.). However, I can't execute the winpty from it for the known error of stdin is not a tty. So the only solution to run standard interactive console Windows tools from the WSL shell is, as I see it, to implement the WSL PTY support in winpty.

Please, can you consider this enhancement?
Thank you!

Note: Regarding the idea of accessing to Win32 name pipes from WSL, the project npiperelaydoes it (using go):
https://github.com/jstarks/npiperelay
https://github.com/tprasadtp/npiperelay

@rprichard
Copy link
Owner

In principle, I think you could ssh from WSL into a Cygwin SSHD, then run winpty.exe from there.

It looks like Microsoft is improving their console/pty/terminal subsystem, and hopefully in the future, the winpty and wslbridge projects won't be needed anymore. See microsoft/terminal#57. It looks like the cmd.exe shell can run inside a WSL Gnome Terminal.

I request why the winpty tool can't implement WSL PTY support (aka "WSL winpty adapter").

I probably could, but I'm not that motivated to work on this.

@lars18th
Copy link
Author

lars18th commented Oct 2, 2018

Hi @rprichard ,

I'll close this issue as the real solution is to implement support for ConPTY API, as described in #152.

I hope you will support it soon. 😉
Regards.

@lars18th lars18th closed this as completed Oct 2, 2018
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