Skip to content

Easier launching of the debugging when /waitForDebugger passed in. #151

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

Conversation

rkeithhill
Copy link
Contributor

OK been doing a lot of debugging of the debug adapater and this has been driving me crazy. Launch VSCode, open PS1 file so I can find the PID of the editor host. Then debug the script, hop over to VS attach to process, find the PID of the second host process (the debug instance) and attach. This new approach will just pop up a dialog box asking which instance of VS to use to debug. You can press NO to skip attaching the debugger and the script will continue to execute.

OK been doing a lot of debugging of the debug adapater and this has been driving me crazy.   Launch VSCode, open PS1 file so I can find the PID of the editor host.  Then debug the script, hop over to VS attach to process, find the PID of the second host process (the debug instance) and attach.  This new approach will just pop up a dialog box asking which instance of VS to use to debug.  You can press NO to skip attaching the debugger and the script will continue to execute.
@rkeithhill
Copy link
Contributor Author

This can be merged independent of the script path spaces fix for #149.

@daviwil
Copy link
Contributor

daviwil commented Feb 10, 2016

This is great, I like it! Don't know why I didn't think of that before ;) If this ends up being a smoother process going forward maybe we can rename the argument to something like /breakOnLaunch or something like that. Not important for now, though.

@daviwil
Copy link
Contributor

daviwil commented Feb 10, 2016

Also, feel free to merge this if you're ready.

rkeithhill added a commit that referenced this pull request Feb 10, 2016
…f-dbghost

Easier launching of the debugging when /waitForDebugger passed in.
@rkeithhill rkeithhill merged commit 5a34236 into PowerShell:master Feb 10, 2016
@rkeithhill rkeithhill deleted the rkeithhill/easier-debugging-of-dbghost branch February 10, 2016 19:01
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