File tree Expand file tree Collapse file tree 4 files changed +4
-7
lines changed Expand file tree Collapse file tree 4 files changed +4
-7
lines changed Original file line number Diff line number Diff line change @@ -4,6 +4,7 @@ A list of all issues and changes can be found in the [v0.28.1 milestone](https:/
44
55### Fixes
66- Skipped launch configuration adjustment to address build errors when debugging using externally launched delve DAP servers. (Issue [ 1793] ( https://github.com/golang/vscode-go/issues/1793 ) )
7+ - Restore the fix for Issue [ 1729] ( https://github.com/golang/vscode-go/issues/1729 ) that was accidentally dropped during merge for release.
78
89## v0.28.0 - 20 Sep, 2021
910
Original file line number Diff line number Diff line change @@ -158,12 +158,6 @@ export class GoDebugConfigurationProvider implements vscode.DebugConfigurationPr
158158 // for local mode, default to dlv-dap.
159159 debugConfiguration [ 'debugAdapter' ] = debugConfiguration [ 'mode' ] !== 'remote' ? 'dlv-dap' : 'legacy' ;
160160 }
161- if ( debugConfiguration [ 'debugAdapter' ] === 'dlv-dap' && debugConfiguration [ 'port' ] ) {
162- this . showWarning (
163- 'ignorePortInDlvDapWarning' ,
164- "`port` is ignored with the 'dlv-dap' debugAdapter, which does not yet support remote debugging. Please file an issue if you have a use case that requires port."
165- ) ;
166- }
167161 if ( debugConfiguration [ 'debugAdapter' ] === 'dlv-dap' && debugConfiguration [ 'mode' ] === 'remote' ) {
168162 this . showWarning (
169163 'ignoreDlvDAPInRemoteModeWarning' ,
Original file line number Diff line number Diff line change @@ -38,6 +38,9 @@ export class GoDebugAdapterDescriptorFactory implements vscode.DebugAdapterDescr
3838 private async createDebugAdapterDescriptorDlvDap (
3939 configuration : vscode . DebugConfiguration
4040 ) : Promise < vscode . ProviderResult < vscode . DebugAdapterDescriptor > > {
41+ if ( configuration . port ) {
42+ return new vscode . DebugAdapterServer ( configuration . port , configuration . host ?? '127.0.0.1' ) ;
43+ }
4144 const logger = new TimestampedLogger ( configuration . trace , this . outputChannel ) ;
4245 logger . debug ( `Config: ${ JSON . stringify ( configuration ) } ` ) ;
4346 const d = new DelveDAPOutputAdapter ( configuration , logger ) ;
Original file line number Diff line number Diff line change @@ -29,7 +29,6 @@ import {
2929 Tool ,
3030 ToolAtVersion
3131} from './goTools' ;
32- import { getFromWorkspaceState } from './stateUtils' ;
3332import {
3433 getBinPath ,
3534 getBinPathWithExplanation ,
You can’t perform that action at this time.
0 commit comments