Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Conversation

@bzoz
Copy link
Contributor

@bzoz bzoz commented Jul 25, 2017

Add a new useWSL boolean attribute to launch arguments. If set to true, debugger will launch Node.js (or given runtimeExecutable) using Windows Subsystem Linux.

When the debugger is instructed to use Linux subsystem it will first test if it is installed. This is done by checking if %WINDIR%\System32\bash.exe (for 64-bit VS Code) or %WINDIR%\SYSNATIVE\bash.exe (for 32-bit VS Code) is present. If found, it is then used as runtimeExecutable, with user-specified runtimeExecutable (with all appropriate parameters) passed as the command to execute (using bash -c parameter). Mapping of the source filenames (the localRoot and remoteRoot attributes) is also automatically adjusted.

For best results, this should be used together with useWSL implementation for the legacy protocol (microsoft/vscode-node-debug#156).

/cc @yodurr

@msftclas
Copy link

@bzoz,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.

It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@msftclas
Copy link

@bzoz, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.

Thanks, Microsoft Pull Request Bot

Add new "useWSL" launch attribute. When used, Bash on Windows will be
used to launch runtimeExecutable. Source files directory mapping will
also be automatically added.
@alanosman
Copy link

Hi - I was scouring the net for this solution too and came across this PR. This is great. How / when do we think we can use this?

@bzoz
Copy link
Contributor Author

bzoz commented Sep 20, 2017

Continued in #142

@bzoz bzoz closed this Sep 20, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants