cmd/go: hide the "TERM" environment variable from "go bug"

Fixes #18128

Change-Id: I5530c9d774a1e398cd759a05bbf308e6e42f0007
Reviewed-on: https://go-review.googlesource.com/33810
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>

diff --git a/src/cmd/go/bug.go b/src/cmd/go/bug.go
index 2977c94c149329f8e689bb14cf39bd169d2b9f2e..cbd258b80bde5f030f5f791ec831e368c89bed6a 100644 (file)
--- a/src/cmd/go/bug.go
+++ b/src/cmd/go/bug.go
@@ -42,7 +42,11 @@ func runBug(cmd *Command, args []string) {
        env := newEnv
        env = append(env, extraEnvVars()...)
        for _, e := range env {
-               fmt.Fprintf(&buf, "%s=\"%s\"\n", e.name, e.value)
+               // Hide the TERM environment variable from "go bug".
+               // See issue #18128
+               if e.name != "TERM" {
+                       fmt.Fprintf(&buf, "%s=\"%s\"\n", e.name, e.value)
+               }
        }
        printGoDetails(&buf)
        printOSDetails(&buf)