Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmd/go: workspace recursive add should skip malformed module definitions #51959

Open
sding3 opened this issue Mar 26, 2022 · 3 comments
Open
Labels
FeatureRequest GoCommand cmd/go modules NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made.
Milestone

Comments

@sding3
Copy link
Contributor

sding3 commented Mar 26, 2022

What version of Go are you using (go version)?

$ go version
go version go1.18 linux/amd64

Does this issue reproduce with the latest release?

Yes

What operating system and processor architecture are you using (go env)?

go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/shang/.cache/go-build"
GOENV="/home/shang/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/shang/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/shang/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.18"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK="/home/shang/go/src/go.work"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build895139390=/tmp/go-build -gno-record-gcc-switches"

What did you do?

mkdir /tmp/reproduce-issue/
cd /tmp/reproduce-issue/
git clone https://go.googlesource.com/tools
go work init
go work use -r .

cat <<EOF > hello.go
package main
import "fmt"
func main() {
    fmt.Println("hello world")
}
EOF

go run hello.go

What did you expect to see?

Expected to see a functional workspace to be created, which should have no effect over the final go run hello.go command.

What did you see instead?

An un-usable workspace was created due to the tools repo containing some purposefully invalid go.mod files meant for unit testing, which go work use -r . added to the workspace, causing the final go run hello.go command to fail with:

go: malformed module path "αfake1α": invalid char 'α'

go work use -r should probably print a warning message for each malformed module definition that it encounters instead of adding those to the workspace.

@sding3 sding3 changed the title cmd/go: cmd/go: workspace should skip malformed module definitions Mar 26, 2022
@sding3 sding3 changed the title cmd/go: workspace should skip malformed module definitions cmd/go: workspace recursive add should skip malformed module definitions Mar 26, 2022
@seankhliao seankhliao added NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. GoCommand cmd/go labels Mar 28, 2022
@seankhliao
Copy link
Member

maybe it should instead be ignoring everything in testdata/ s

cc @bcmills @matloob

@sding3
Copy link
Contributor Author

sding3 commented Mar 28, 2022

Another approach, orthogonal to your ignore testdata/ suggestion, is to add a -maxdepth <int> flag to the go work use -r command to limit the search scope of the recursive directory descend.

@bcmills
Copy link
Contributor

bcmills commented May 9, 2022

go.mod files in testdata directories prune that testdata out of the resulting module (see also #27852), which will likely either cause the test to become a no-op (and thus miss real bugs) or cause the test to always fail (and produce spurious failures in, say, go test all that can be a nuisance to filter out).

For the foreseeable future, tests which require their own go.mod file will need to construct the module within a temp directory instead of relying on it in a testdata subdirectory.

@bcmills bcmills added FeatureRequest modules NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. labels May 11, 2022
@bcmills bcmills added this to the Backlog milestone May 11, 2022
@gopherbot gopherbot removed the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FeatureRequest GoCommand cmd/go modules NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made.
Projects
None yet
Development

No branches or pull requests

4 participants