diff options
| author | Robert Griesemer <gri@google.com> | 2023-05-24 16:30:49 +0000 |
|---|---|---|
| committer | Gopher Robot <gobot@golang.org> | 2023-05-24 16:46:45 +0000 |
| commit | 42f387aae123ec416b57d84ba96ec4a5ef0e8a1b (patch) | |
| tree | d738598ec729501a5efe70e082154399b1ed7a7f /src | |
| parent | 0adcc5ace8c01d8bf587827e672120d74fbaca36 (diff) | |
| download | go-42f387aae123ec416b57d84ba96ec4a5ef0e8a1b.tar.xz | |
Revert "cmd/compile: enable printing of error URLs by default"
This reverts commit e7a9ca0a53db4b4f357b5d3caaef65e44ce08a90.
Reason for revert: Decided to delay to Go 1.22.
Change-Id: I4635cb4c1372b54cac573041be8a43e294de5183
Reviewed-on: https://go-review.googlesource.com/c/go/+/497975
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Robert Griesemer <gri@google.com>
Reviewed-by: Russ Cox <rsc@golang.org>
Auto-Submit: Robert Griesemer <gri@google.com>
Reviewed-by: Robert Griesemer <gri@google.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/cmd/compile/internal/base/flag.go | 1 | ||||
| -rw-r--r-- | src/cmd/internal/testdir/testdir_test.go | 6 |
2 files changed, 3 insertions, 4 deletions
diff --git a/src/cmd/compile/internal/base/flag.go b/src/cmd/compile/internal/base/flag.go index a5441ffac2..753a60ae1e 100644 --- a/src/cmd/compile/internal/base/flag.go +++ b/src/cmd/compile/internal/base/flag.go @@ -164,7 +164,6 @@ func ParseFlags() { Flag.LinkShared = &Ctxt.Flag_linkshared Flag.Shared = &Ctxt.Flag_shared Flag.WB = true - Flag.ErrorURL = true Debug.ConcurrentOk = true Debug.InlFuncsWithClosures = 1 diff --git a/src/cmd/internal/testdir/testdir_test.go b/src/cmd/internal/testdir/testdir_test.go index d9f78f2f6a..c86cc20a9c 100644 --- a/src/cmd/internal/testdir/testdir_test.go +++ b/src/cmd/internal/testdir/testdir_test.go @@ -184,7 +184,7 @@ func compileInDir(runcmd runCmd, dir string, flags []string, importcfg string, p if importcfg == "" { importcfg = stdlibImportcfgFile() } - cmd := []string{goTool, "tool", "compile", "-e", "-D", "test", "-errorurl=false", "-importcfg=" + importcfg} + cmd := []string{goTool, "tool", "compile", "-e", "-D", "test", "-importcfg=" + importcfg} if pkgname == "main" { cmd = append(cmd, "-p=main") } else { @@ -201,7 +201,7 @@ func compileInDir(runcmd runCmd, dir string, flags []string, importcfg string, p return runcmd(cmd...) } -var stdlibImportcfgStringOnce sync.Once // TODO(#56102): Use sync.OnceValue once available. Also below. +var stdlibImportcfgStringOnce sync.Once // TODO(#56102): Use sync.OnceValue once availabe. Also below. var stdlibImportcfgString string func stdlibImportcfg() string { @@ -745,7 +745,7 @@ func (t test) run() error { // Fail if wantError is true and compilation was successful and vice versa. // Match errors produced by gc against errors in comments. // TODO(gri) remove need for -C (disable printing of columns in error messages) - cmdline := []string{goTool, "tool", "compile", "-p=p", "-d=panic", "-C", "-e", "-errorurl=false", "-importcfg=" + stdlibImportcfgFile(), "-o", "a.o"} + cmdline := []string{goTool, "tool", "compile", "-p=p", "-d=panic", "-C", "-e", "-importcfg=" + stdlibImportcfgFile(), "-o", "a.o"} // No need to add -dynlink even if linkshared if we're just checking for errors... cmdline = append(cmdline, flags...) cmdline = append(cmdline, long) |