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: build fails in Plan 9 under /n #27074

Closed
fhs opened this issue Aug 18, 2018 · 2 comments

Comments

Projects
None yet
3 participants
@fhs
Copy link
Contributor

commented Aug 18, 2018

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

master (64fae25)

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

plan9/amd64

What did you do?

cpu% cd /n/go-plan9/src
cpu% ./all.rc

What did you expect to see?

successful build

What did you see instead?

Building Go cmd/dist using /mnt/term/home/big/go-plan9-amd64-bootstrap
Building Go toolchain1 using /mnt/term/home/big/go-plan9-amd64-bootstrap.
Building Go bootstrap cmd/go (go_bootstrap) using Go toolchain1.
Building Go toolchain2 using go_bootstrap and Go toolchain1.
go: cannot determine module path for source directory /n (outside GOPATH, no import comments)
go tool dist: FAILED: /n/go-plan9/pkg/tool/plan9_amd64/go_bootstrap install -gcflags=all= -ldflags=all= -i cmd/asm cmd/cgo cmd/compile cmd/link: exit status: 'go_bootstrap 1770: 1'

It's failing because go command found /n/go.mod when looking for a go.mod file, but it's an automatically created empty directory. In Plan 9, directories in /n are automatically created when accessed. Go command should check /n/go.mod is not a directory. I will send CL soon.

@gopherbot

This comment has been minimized.

Copy link

commented Aug 18, 2018

Change https://golang.org/cl/129804 mentions this issue: cmd/go/internal/modload: ignore directories when looking for go.mod file

@fhs fhs changed the title Module build fails in Plan 9 under /n cmd/go: build fails in Plan 9 under /n Aug 18, 2018

@bradfitz bradfitz added the OS-Plan9 label Aug 20, 2018

@bradfitz bradfitz added this to the Go1.12 milestone Aug 20, 2018

@bradfitz

This comment has been minimized.

Copy link
Member

commented Aug 20, 2018

/cc @0intro

@gopherbot gopherbot closed this in e8f49aa Aug 20, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.