]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/go: clean up "cover" debug output
authorAustin Clements <austin@google.com>
Wed, 11 Oct 2023 16:20:19 +0000 (12:20 -0400)
committerGopher Robot <gobot@golang.org>
Wed, 11 Oct 2023 16:39:44 +0000 (16:39 +0000)
Currently, the two places that run the cover tool manually construct a
terse command description. However, this actually prevents Builder.run
from constructing a more detailed description from the actual command
being run. Fix this by passing "" as the description to get the
default behavior.

Change-Id: I27d42cb1fda9bba70c631dc43417a03b8bddec92
Reviewed-on: https://go-review.googlesource.com/c/go/+/534157
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Auto-Submit: Austin Clements <austin@google.com>
Reviewed-by: Than McIntosh <thanm@google.com>
src/cmd/go/internal/work/exec.go

index 0a9e109c50ad2cf29bed23c91261698c645767e3..42ecac6d03347a56aa3489717c5606b2b8b6e976 100644 (file)
@@ -2036,7 +2036,7 @@ func (b *Builder) installHeader(ctx context.Context, a *Action) error {
 //
 //     go tool cover -mode=b.coverMode -var="varName" -o dst.go src.go
 func (b *Builder) cover(a *Action, dst, src string, varName string) error {
-       return b.run(a, a.Objdir, "cover "+a.Package.ImportPath, nil,
+       return b.run(a, a.Objdir, "", nil,
                cfg.BuildToolexec,
                base.Tool("cover"),
                "-mode", a.Package.Internal.Cover.Mode,
@@ -2069,7 +2069,7 @@ func (b *Builder) cover2(a *Action, infiles, outfiles []string, varName string,
                "-outfilelist", covoutputs,
        }
        args = append(args, infiles...)
-       if err := b.run(a, a.Objdir, "cover "+a.Package.ImportPath, nil,
+       if err := b.run(a, a.Objdir, "", nil,
                cfg.BuildToolexec, args); err != nil {
                return nil, err
        }