Skip to content

Commit 3c8da71

Browse files
committed
[release] src/debugAdapter/goDebug: do not log env vars additionally
We added this logging while trying to help users inspect env var propagation when using the legacy debug adapter. All the env vars are included as a launch parameter (`env`), so instead users can inspect the DAP trace and this is unnecessary. Remove this - this also pollutes the test logging. Change-Id: I25caeccb1f189291bb662829f544357d419050ef Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/349897 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]> (cherry picked from commit d5bd4eb) Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/350136
1 parent e3a7a49 commit 3c8da71

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

src/debugAdapter/goDebug.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -506,11 +506,6 @@ export class Delve {
506506
log(`Using GOPATH: ${env['GOPATH']}`);
507507
log(`Using GOROOT: ${this.goroot}`);
508508
log(`Using PATH: ${env['PATH']}`);
509-
if (launchArgs.trace === 'verbose') {
510-
Object.keys(env).forEach((key) => {
511-
log(' export ' + key + '="' + env[key] + '"');
512-
});
513-
}
514509
if (launchArgs.noDebug) {
515510
if (mode === 'debug') {
516511
this.noDebug = true;

0 commit comments

Comments
 (0)