Skip to content
This repository has been archived by the owner on Jul 15, 2023. It is now read-only.

Add note about needing WSL 2 on Windows 10 #3167

Merged
merged 2 commits into from
Apr 18, 2020

Conversation

mpatnode
Copy link
Contributor

@mpatnode mpatnode commented Apr 9, 2020

delve will hang on the WSL 1 kernel due to trace system call issues. Just adding a note to that effect. Would be nice to put in the Wiki under troubleshooting as well. @ramya-rao-a

Copy link
Contributor

@ramya-rao-a ramya-rao-a left a comment

Choose a reason for hiding this comment

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

Looks like this is your first PR contribution to this project, Thanks & Welcome @mpatnode!

We just copied over all the content from the wiki to the docs folder in this repo and will be updating the wiki to just point to these.

Can you make a similar update in the wiki like you mentioned?

@@ -207,7 +207,7 @@ A few of these are available in the editor context menu as an experimental featu

### _Optional_: Debugging

To use the debugger, you must currently manually install `delve`. See the [Installation Instructions](https://github.com/derekparker/delve/tree/master/Documentation/installation) for full details. On MacOS it requires creating a self-signed cert to sign the `dlv` binary.
To use the debugger, you must currently manually install `delve`. See the [Installation Instructions](https://github.com/derekparker/delve/tree/master/Documentation/installation) for full details. On MacOS it requires creating a self-signed cert to sign the `dlv` binary. If using WSL on Windows, you will need the WSL 2 Linux kernel. See [WSL 2 Installation](https://docs.microsoft.com/en-us/windows/wsl/wsl2-install) and note the Window 10 build version requirements.
Copy link
Contributor

Choose a reason for hiding this comment

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

and note the Window 10 build version requirements.

Its not clear to me as to why we want to call this out. Can you elaborate?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The default WSL uses syscall mapping, hence delve just hangs on the trace() call. You need to join the Windows Insiders program and get the latest version of the OS, and then you can install the WSL 2 kernel, which is a true virtualized machine.

@mpatnode
Copy link
Contributor Author

We just copied over all the content from the wiki to the docs folder in this repo and will be updating the wiki to just point to these.

Can you make a similar update in the wiki like you mentioned?

👍 will do

@msftclas
Copy link

msftclas commented Apr 16, 2020

CLA assistant check
All CLA requirements met.

@mpatnode mpatnode force-pushed the master branch 2 times, most recently from 7d9a2be to 048f0e0 Compare April 16, 2020 19:23
Try running ```delve debug ./main``` at the WSL command line and see if you get a prompt

**_Solution_**: Ensure you are running the WSL 2 Kernel, which (as of 4/15/2020) requires an early release of the Windows 10 OS. This is available to anyone via the Windows Insider program. See [WSL 2 Installation](https://docs.microsoft.com/en-us/windows/wsl/wsl2-install)

#### could not launch process: could not fork/exec
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ramya-rao-a Wasn't sure what to do with this dangler.

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry, which dangler?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The "could not launch process: could not fork/exec" line has no explanation or solution

Copy link
Contributor

@ramya-rao-a ramya-rao-a Apr 18, 2020

Choose a reason for hiding this comment

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

Ah! The solution to that is split for Mac vs Linux, looks like the lack of indentation isnt helping anyone :) I'll fix that

Copy link
Contributor

@ramya-rao-a ramya-rao-a left a comment

Choose a reason for hiding this comment

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

Thanks!

@ramya-rao-a ramya-rao-a merged commit ffc1a56 into microsoft:master Apr 18, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants