From 612469ab0b57bb53d1f054e37b64a63c08d8c6db Mon Sep 17 00:00:00 2001 From: Mohit Agarwal Date: Thu, 1 Dec 2016 23:20:03 +0530 Subject: [PATCH] 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 Run-TryBot: Brad Fitzpatrick TryBot-Result: Gobot Gobot --- src/cmd/go/bug.go | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/cmd/go/bug.go b/src/cmd/go/bug.go index 2977c94c14..cbd258b80b 100644 --- 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) -- 2.50.0