Skip to content

Commit ab3d3a6

Browse files
committed
src/goDebugFactory: add --check-go-version=false
If go-delve/delve#2684 is in, users who use go versions outside delve's officially supported version range, will have visible warnings. Otherwise, we will need to have our own custom version check and make the warning visible. Fixes golang#1716 Change-Id: Ide76978b038f4efecdf3485abdcae37323855af5 Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/347562 Trust: Hyang-Ah Hana Kim <[email protected]> Run-TryBot: Hyang-Ah Hana Kim <[email protected]> TryBot-Result: kokoro <[email protected]> Reviewed-by: Suzy Mueller <[email protected]>
1 parent 9a9ec9a commit ab3d3a6

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

src/goDebugFactory.ts

+4-1
Original file line numberDiff line numberDiff line change
@@ -367,8 +367,11 @@ function spawnDlvDapServerProcess(
367367

368368
const dlvArgs = new Array<string>();
369369
dlvArgs.push('dap');
370-
// add user-specified dlv flags first. When duplicate flags are specified,
370+
// When duplicate flags are specified,
371371
// dlv doesn't mind but accepts the last flag value.
372+
// Add user-specified dlv flags first except
373+
// --check-go-version that we want to disable by default but allow users to override.
374+
dlvArgs.push('--check-go-version=false');
372375
if (launchAttachArgs.dlvFlags && launchAttachArgs.dlvFlags.length > 0) {
373376
dlvArgs.push(...launchAttachArgs.dlvFlags);
374377
}

0 commit comments

Comments
 (0)