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

Add a workaround to support VS Code's Remote WSL extension (fix #238) #324

Closed
wants to merge 1 commit into from

Conversation

Atry
Copy link
Contributor

@Atry Atry commented Nov 3, 2023

This PR adds modules/vscode-remote-wsl-extension-workaround.nix, when wsl.vscodeRemoteWslExtensionWorkaround.enable is true, VS Code's Remote WSL extension can connect to NixOS, fixing #238

See https://github.com/Atry/nixos-wsl-vscode/blob/main/flake.nix for usage

@Atry Atry changed the title Add modules/vscode-remote-wsl-extension-workaround.nix (fix #238) Add a workaround to support VS Code's Remote WSL extension (fix #238) Nov 3, 2023
@Atry Atry changed the title Add a workaround to support VS Code's Remote WSL extension (fix #238) Add a workaround to support VS Code's Remote WSL extension (fix #222 and #238) Nov 3, 2023
@Atry Atry changed the title Add a workaround to support VS Code's Remote WSL extension (fix #222 and #238) Add a workaround to support VS Code's Remote WSL extension (fix #238) Nov 3, 2023
@K900
Copy link
Contributor

K900 commented Nov 3, 2023

I would prefer to see something like https://github.com/K900/vscode-remote-workaround, which simply uses nixpkgs node.js

@Atry
Copy link
Contributor Author

Atry commented Nov 8, 2023

This PR is technically not a workaround. It just provides what vscode-server assumes (global C runtime and global coreutils).

@K900
Copy link
Contributor

K900 commented Nov 8, 2023

Then neither is my approach, because it provides a functioning Node runtime.

@Atry
Copy link
Contributor Author

Atry commented Nov 8, 2023 via email

@K900
Copy link
Contributor

K900 commented Nov 8, 2023

Correct.

Copy link
Member

@SuperSandro2000 SuperSandro2000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this affects every program and can cause unexpected regressions I am very reluctant to accept it, especially regarding that we would still need an older WSL release which is probably a separate problem but is helping the situation not.

description = "Workaround for VSCode's Remote WSL extension";
};
nix-ld-rs = lib.mkOption {
type = lib.types.raw;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be type package.

@Atry
Copy link
Contributor Author

Atry commented Nov 10, 2023

Since you are reluctant to accept this PR. I just put it in a separate repository and close this PR.

Feel free to use this module from https://github.com/Atry/nixos-wsl-vscode

@Atry Atry closed this Nov 10, 2023
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

Successfully merging this pull request may close these issues.

3 participants