diff options
author | Bryan C. Mills <bcmills@google.com> | 2021-06-15 12:05:01 -0400 |
---|---|---|
committer | Bryan C. Mills <bcmills@google.com> | 2021-06-16 20:38:07 +0000 |
commit | 0e67ce3d28320e816dd8e7cf7d701c1804fb977e (patch) | |
tree | fd02890576aa168e1d67423b22ad12de09e4fed9 /src/cmd/go/internal/modload/init.go | |
parent | 6ea2af0890260fec6cc951b5f426c0464e43266d (diff) | |
download | go-0e67ce3d28320e816dd8e7cf7d701c1804fb977e.tar.gz go-0e67ce3d28320e816dd8e7cf7d701c1804fb977e.zip |
cmd/go: in lazy modules, add transitive imports for 'go get' arguments
I needed to also update TestScript/mod_sumdb_golang.
It had been relying on 'go list -mod=mod' to add both the go.mod and
go.sum entries for the named package, but when 'go get' actually adds
all of the needed dependencies, lazy loading kicks in and 'go list'
doesn't end up needing the checksums for go.mod files.
We didn't detect the skew before because the 'go list' command was
(unexpectedly) also adding the missing dependencies, which triggered a
deep scan of the complete module graph.
For #45979
Change-Id: Ica917dee22c83ffa71c6ad0f2e189f911b73edf4
Reviewed-on: https://go-review.googlesource.com/c/go/+/328231
Trust: Bryan C. Mills <bcmills@google.com>
Run-TryBot: Bryan C. Mills <bcmills@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Michael Matloob <matloob@golang.org>
Diffstat (limited to 'src/cmd/go/internal/modload/init.go')
-rw-r--r-- | src/cmd/go/internal/modload/init.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/cmd/go/internal/modload/init.go b/src/cmd/go/internal/modload/init.go index eb9cfe629b..cbc7289afa 100644 --- a/src/cmd/go/internal/modload/init.go +++ b/src/cmd/go/internal/modload/init.go @@ -661,7 +661,7 @@ func requirementsFromModFile(ctx context.Context) *Requirements { for _, n := range mPathCount { if n > 1 { var err error - rs, err = updateRoots(ctx, rs.direct, rs, nil, nil) + rs, err = updateRoots(ctx, rs.direct, rs, nil, nil, false) if err != nil { base.Fatalf("go: %v", err) } |