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

Newly installed debug extension does not work #69100

Closed
actboy168 opened this issue Feb 21, 2019 · 10 comments
Closed

Newly installed debug extension does not work #69100

actboy168 opened this issue Feb 21, 2019 · 10 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug debug Debug viewlet, configurations, breakpoints, adapter issues verified Verification succeeded
Milestone

Comments

@actboy168
Copy link

Previous #69089

Please don't let me contact the author of the extension because he is already here.

It seems that the hot reload extension is not working correctly. Please tell me how to fix it. If it needs the extension to modify anything, thanks.

@vscodebot vscodebot bot added the extensions Issues concerning extensions label Feb 21, 2019
@sandy081 sandy081 added the info-needed Issue requires more information from poster label Feb 21, 2019
@sandy081
Copy link
Member

@actboy168 Can you please explain what is not working with steps to reproduce.

@actboy168
Copy link
Author

1.Open a directory
2.New file test.lua
3.Type print 'hello'
4.Install extension actboy168.lua-debug
5.F5

It will get an error message.

Cannot find debug adapter for type 'lua'.

@sandy081 sandy081 assigned weinand and unassigned sandy081 Feb 22, 2019
@sandy081 sandy081 removed the extensions Issues concerning extensions label Feb 22, 2019
@weinand weinand assigned isidorn and unassigned weinand Feb 22, 2019
@weinand weinand removed the info-needed Issue requires more information from poster label Feb 22, 2019
@weinand
Copy link
Contributor

weinand commented Feb 22, 2019

@actboy168 what version of VS Code are you using?

@isidorn @alexandrudima if the lua debugger hsn't been upgrade to dynamic install, installing it should still ask for a reload, right?

@weinand weinand added the info-needed Issue requires more information from poster label Feb 22, 2019
@actboy168
Copy link
Author

@weinand
VS Code version: Code 1.31.1 (1b8e830, 2019-02-12T02:20:54.427Z)

@isidorn
Copy link
Contributor

isidorn commented Feb 22, 2019

The issue is comming from here https://github.com/Microsoft/vscode/blob/275d2a02ca922738b58cea7dbf9dce229ea75f21/src/vs/workbench/contrib/debug/node/debugger.ts#L92

For my machine the issue is that the debug adapters are set to run in EH and the debug adapter factories are not aware of the lua debug type.
Forwarding to @weinand since he introduced the debug adapter factories.

Here's the code pointer where we get pinged from the extension service that a new extesion is installed. It might need to aslo properly update the factories https://github.com/Microsoft/vscode/blob/275d2a02ca922738b58cea7dbf9dce229ea75f21/src/vs/workbench/contrib/debug/electron-browser/debugConfigurationManager.ts#L241

@isidorn isidorn assigned weinand and unassigned isidorn Feb 22, 2019
@isidorn isidorn added debug Debug viewlet, configurations, breakpoints, adapter issues and removed info-needed Issue requires more information from poster labels Feb 22, 2019
@weinand
Copy link
Contributor

weinand commented Feb 22, 2019

@isidorn Why are debug adapters are set to run in EH in your case? Do you have LiveShare installed or do you use the setting that forces the DA to run in EH?

@weinand
Copy link
Contributor

weinand commented Feb 22, 2019

Oh I see. The lua extension uses the tracker.

@weinand
Copy link
Contributor

weinand commented Feb 22, 2019

@actboy168 the only way to avoid this problem is by not registering a DebugAdapterTracker in the case that the "lua" extension got installed via hot reload.

We are looking into fixing this on our end.

@weinand
Copy link
Contributor

weinand commented Feb 22, 2019

I have a fix which eliminates the error message "Cannot find debug adapter for type 'lua'.".

But F5 always results in this following error:
2019-02-22_22-56-54

Since this error does not go away after a reload, I suspect that lua is not working on my macOS anyway.

@weinand
Copy link
Contributor

weinand commented Feb 22, 2019

@actboy168 please try the Insiders from next Monday.

@weinand weinand added this to the February 2019 milestone Feb 22, 2019
@weinand weinand added the bug Issue identified by VS Code Team member as probable bug label Feb 22, 2019
@roblourens roblourens added the verified Verification succeeded label Feb 27, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Apr 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug debug Debug viewlet, configurations, breakpoints, adapter issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants