diff options
| author | Russ Cox <rsc@golang.org> | 2018-07-06 01:29:47 -0400 |
|---|---|---|
| committer | Russ Cox <rsc@golang.org> | 2018-07-10 03:56:55 +0000 |
| commit | fd263ccefe4f1280b357cb1cb481a6599efd5f6b (patch) | |
| tree | 03a2309c756042e273c4a857c992f811e94c0471 /src/net | |
| parent | 5a8b652c6e93311fb6f336e7075aee46ef560213 (diff) | |
| download | go-fd263ccefe4f1280b357cb1cb481a6599efd5f6b.tar.xz | |
cmd/cover: invoke go command to find packages
cmd/cover has always assumed that package x/y/z can be
found in $GOPATH/src/x/y/z (roughly; by using go/build).
That won't be true for too much longer. Instead, run the
go command to find out where packages are.
This will make 'go tool cover' safe for use with Go modules
when they are in use in Go 1.11, and it continues to work
with the existing Go toolchains too.
An alternative would be to modify the cover profile format
to record file names directly, but that would require also
updating golang.org/x/tools/cover/profile and any tools
that use it, which seems not worth the trouble.
(That fork of the code does not contain any code to resolve
package names to directory locations, so it's unaffected.)
No new test here: cmd/go's TestCoverageFunc tests this code.
Fixes #25318 (when people use Go 1.11 instead of vgo).
Change-Id: I8769b15107aecf25f7aaf8692b724cf7d0f073d0
Reviewed-on: https://go-review.googlesource.com/122478
Run-TryBot: Russ Cox <rsc@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Alan Donovan <adonovan@google.com>
Diffstat (limited to 'src/net')
0 files changed, 0 insertions, 0 deletions
