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.

Sign up for GitHub

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

Jump to bottom

Attach to Process + Node v0.12.6 (Linux) fails to detect node version #21231

Closed
mousetraps opened this issue Feb 22, 2017 · 2 comments
Closed

Attach to Process + Node v0.12.6 (Linux) fails to detect node version #21231

mousetraps opened this issue Feb 22, 2017 · 2 comments
Assignees
@roblourens @weinand
Labels
debug Debug viewlet, configurations, breakpoints, adapter issues
Milestone
February 2017

Comments

@mousetraps
Copy link
Contributor

Testing #20991
screenshot from 2017-02-22 18-38-23

And yet, for some reason it works fine with the Launch Program or Attach to Program configuration
@roblourens
Copy link
Member

The message is technically true, but not that useful. We are assuming legacy protocol, because it's not using the inspector protocol. This sounds like an error, though. "Debugging with legacy protocol because inspector protocol was not detected"?

@weinand weinand added the debug Debug viewlet, configurations, breakpoints, adapter issues label Feb 23, 2017
@weinand weinand added this to the February 2017 milestone Feb 23, 2017
@weinand
Copy link
Contributor

weinand commented Feb 23, 2017

For the processId case now showing this message:
"Debugging with legacy protocol because attaching to a process by ID is only supported for legacy protocol."

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@roblourens roblourens

@weinand weinand

Labels
debug Debug viewlet, configurations, breakpoints, adapter issues
Projects
None yet
Milestone
February 2017
Development

No branches or pull requests
3 participants
@roblourens @mousetraps @weinand