Related Issues and Documentation

• x/pkgsite: cannot refresh documentation locally #54479 (closed)
• x/pkgsite/cmd/pkgsite: templates aren't embedded, causing an error if run from another directory #48410 (closed)
• x/pkgsite/cmd/pkgsite: Unable to fetch std@Latest Pkgsite documentation for Local Main go Repo : https://go.googlesource.com/go #63744
• x/pkgsite/cmd/frontend: no longer loads stdlib in proxy mode #63249
• x/pkgsite@latest doesn't install #54672 (closed)
• x/pkgsite: go get golang.org/x/pkgsite fails when run from pkgsite root folder #66702 (closed)
• cmd/go: tests fail when GOROOT is stale #48698 (closed)
• x/pkgsite: start localserver Pkg.go.dev fail #54692 (closed)
• x/pkgsite/cmd/pkgsite: panic for "multiple registrations" when running locally #64903 (closed)
• godoc does not work for local modules #45083 (closed)

_{(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)}

I believe this is a dupe of #57742

@seankhliao I read that issue, and it caused me to try a few other things I didn't mention above, such as -list=false and cding to GOROOT/src and ~/apps/godev/src first, but none of them worked.

I assume that @matloob did fix something, so I interpreted that as meaning that there's some subtlety about what #57742 is about that's different from this issue. But I'm also okay with just declaring that #57742 was never fixed and reopening that one and closing this one.

Just this (without -gorepo) seems to work?

cd $(go env GOROOT)/src
pkgsite .

@seankhliao I do not want to serve files from GOROOT. I want to serve the files out of my local repo checkout in ~/apps/godev.

One thing I tried is to set GOROOT to be ~/apps/godev, but that also doesn't work:

$ cd ~/apps/godev/src/
$ GOROOT=$(echo ~/apps/godev) pkgsite .
searching GOPATH: listing modules in .: running go with ["list" "-m" "-json"]: exit status 1:

(Another bad error message: it again mentions GOPATH, and it ends with a colon and no further text.)

In any case, I want what the -gorepo flag says it does, but it does not work. If there is some other way point pkgsite at my local checkout of the Go repo, then we should (a) make it work well, (b) document it, and (c) remove the -gorepo flag.

does the go list command work in your ~/apps/godev checkout? or do you need to adjust your PATH to ensure the go binary used is the correct one?

@seankhliao sure, if I build a go binary in my repo checkout and then override both PATH and GOROOT (effectively changing what the "installed" go is), I can get the tool to use the checkout. -gorepo is not even involved then:

$ PATH="$(echo ~/apps/godev)/bin:$PATH" GOROOT="$(echo ~/apps/godev)" pkgsite .
2024/07/21 11:47:15 Info: go/packages.Load(["all"]) loaded 305 packages from . in 156.055794ms
2024/07/21 11:47:15 Info: go/packages.Load(std) loaded 305 packages from /home/caleb/apps/godev in 243.957558ms

@cespare Thanks for reporting this! We recognize that cmd/pkgsite's user experience is not quite there yet and appreciate you bearing with us.

There's a bug where we print searching GOPATH in the error message even in the non-GOPATH branch of an if statement.

There's a bug where we use the value of the gorepo flag before the flag is parsed. I'll send a change to fix that.

I wasn't able to get the error in #68533 (comment) but I can look into that once we fix the first two issues.

I think we should also probably remove the -gorepo flag from cmd/pkgsite. I think it's confusing to have multiple ways of telling go where your standard library is. In the rest of our tools the way to get information about standard library packages from a specific GOROOT is to set PATH so that the go command run was built from that GOROOT.

Change https://go.dev/cl/600917 mentions this issue: cmd/pkgsite: fix setting gorepo flag

@matloob Can you close this issue if cl/600917 fixes the gorepo flag?