-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
VS Code debuggers using resolveDebugConfigurationWithSubstitutedVariables do not launch #7690
Comments
True, we don't support this API yet. Relevant PR in VS Code: microsoft/vscode@dba0076 PRs are welcomed. |
…tutedVariables VS Code API Signed-off-by: Anton Kosyakov <anton.kosyakov@typefox.io>
@akosyakov is there an easy way to tell when a change is included in production GitPod? I'm going to build an updated Dart extension today, and I'm trying to figure out if I still need a workaround for this. |
@DanTup I'm planning to update Gitpod staging today. Do you have access to it? I can let you know. I don't know when it will make to production. cc @svenefftinge |
Yep, I think I do still have access to staging. The question was mainly whether I needed the workaround for production though - if it's not already live, I'll include it (I'm building a specific copy of the extension to make my GitPod/Flutter web sample work again with the latest release of Dart-Code for those that want to try it out). Thanks! |
…tutedVariables VS Code API Signed-off-by: Anton Kosyakov <anton.kosyakov@typefox.io>
@akosyakov what's the status of this - were you waiting for me to test on staging? I tried on live today, but this doesn't seem to be working ( |
I'm trying to track down an issue with debug sessions in the Dart VS Code extension not launching, and my suspicion is that it's due to us switching to using
resolveDebugConfigurationWithSubstitutedVariables
(see microsoft/vscode#87450).VS Code now calls
resolveDebugConfiguration
as it previously did, then it resolves any variables (for example${file}
,${workspaceFolder}
etc.) and then callsresolveDebugConfigurationWithSubstitutedVariables
. I think the code doing this is https://github.com/microsoft/vscode/blob/921b9a92a1b662fbe3aa21f47b0f8ed1fd89c947/src/vs/workbench/contrib/debug/browser/debugService.ts#L672-L685.The text was updated successfully, but these errors were encountered: