From ef3bb638de442dc41d1e34a32e893d2041731e8c Mon Sep 17 00:00:00 2001 From: Michael Matloob Date: Thu, 22 May 2025 09:12:48 -0700 Subject: [PATCH] Revert "cmd/doc: better support for no network" This reverts commit 988eb0d11e8d96e8ca150f401ed82326b276f653. Reason for revert: breaks viewing documentation for unfetched modules For #73833 Change-Id: I89bc459e820c85e96837d1707058501488a14eef Reviewed-on: https://go-review.googlesource.com/c/go/+/675575 Reviewed-by: Jonathan Amsterdam LUCI-TryBot-Result: Go LUCI Reviewed-by: Michael Matloob --- src/cmd/doc/main.go | 10 +--------- src/cmd/go/internal/load/pkg.go | 2 +- 2 files changed, 2 insertions(+), 10 deletions(-) diff --git a/src/cmd/doc/main.go b/src/cmd/doc/main.go index 22db39ecc9..ccd8512006 100644 --- a/src/cmd/doc/main.go +++ b/src/cmd/doc/main.go @@ -259,18 +259,10 @@ func doPkgsite(urlPath string) error { signal.Ignore(signalsToIgnore...) const version = "v0.0.0-20250520201116-40659211760d" - docatversion := "golang.org/x/pkgsite/cmd/internal/doc@" + version - // First download the module and then try to run with GOPROXY=off to circumvent - // the deprecation check. This will allow the pkgsite command to run if it's - // in the module cache but there's no network. - if _, err := runCmd(nil, "go", "mod", "download", docatversion); err != nil { - return err - } - cmd := exec.Command("go", "run", docatversion, + cmd := exec.Command("go", "run", "golang.org/x/pkgsite/cmd/internal/doc@"+version, "-gorepo", buildCtx.GOROOT, "-http", addr, "-open", path) - cmd.Env = append(os.Environ(), "GOPROXY=off") cmd.Stdout = os.Stderr cmd.Stderr = os.Stderr diff --git a/src/cmd/go/internal/load/pkg.go b/src/cmd/go/internal/load/pkg.go index 34e8c90b2f..e913f98852 100644 --- a/src/cmd/go/internal/load/pkg.go +++ b/src/cmd/go/internal/load/pkg.go @@ -3410,7 +3410,7 @@ func PackagesAndErrorsOutsideModule(ctx context.Context, opts PackageOpts, args } rootMod := qrs[0].Mod deprecation, err := modload.CheckDeprecation(ctx, rootMod) - if err != nil && !errors.Is(err, fs.ErrNotExist) { + if err != nil { return nil, fmt.Errorf("%s: %w", args[0], err) } if deprecation != "" { -- 2.50.0