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

Look into leveraging vscode.env.shell in remote workspaces #20

Open
Tyriar opened this issue Jun 8, 2020 · 3 comments
Open

Look into leveraging vscode.env.shell in remote workspaces #20

Tyriar opened this issue Jun 8, 2020 · 3 comments
Labels
enhancement New feature or request

Comments

@Tyriar
Copy link
Owner

Tyriar commented Jun 8, 2020

Currently the SSH support works by ignoring which shell you selected and opening the host with the default shell. Perhaps instead we should pull vscode.env.shell which should be the default remote shell and make that the default and potentially remove the other options?

FYI @ricardosantos9521

@Tyriar Tyriar added the enhancement New feature or request label Jun 8, 2020
@ricardosantos9521
Copy link
Contributor

ricardosantos9521 commented Jun 8, 2020

Yes Windows terminal is using the default shell which can be a problem sometimes. Disabling selection would be the best scenario. We should try to use vscode.env.shell as the default windows terminal shell if we can otherwise try to use the default.

In the remote we should use vscode.env.shell if the remote is windows otherwise use we should use what we have already implemented.
Can we get the default shell of the remote workspace through vscode api when we are a ui extension?

@Tyriar
Copy link
Owner Author

Tyriar commented Jun 8, 2020

Can we get the default shell of the remote workspace through vscode api when we are a ui extension?

Yep, that's what vscode.env.shell will do. It is not possible to pull /etc/shells with SSH though without making our own connection, we could possibly do it if the SSH extension wanted to expose an API to do it. Maybe we should implement the remote shell detection in WSL first and if it works nicely explore options for SSH?

@ricardosantos9521
Copy link
Contributor

vscode.env.shell returns always the terminal.integrated.shell.windows so i think we should use terminal.integrated.shell.windows and terminal.integrated.shell.linux as shells for
wsl and ssh

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

No branches or pull requests

2 participants