cmd/go: ignore directories under goroot/src

[alberts]

What steps will reproduce the problem?

Make a tree with src/cmd/main.go:

$ cat src/cmd/main.go
package main
func main() {}

What is the expected output?

a command called cmd

What do you see instead?

GOPATH=/path/to/proj go list ./...
_/path/to/proj/src/cmd

GOPATH=/path/to/proj go install -v ./...
_/path/to/proj/src/cmd

this go install also doesn't actually install anything to the bin directory.

If you run go install -x you can see that it links an a.out but after that nothing
happens.

Which compiler are you using (5g, 6g, 8g, gccgo)?

6g

Which operating system are you using?

linux

Which revision are you using?  (hg identify)

go version weekly.2012-03-22 +ee5c7588cb7f

Please provide any additional information below.

[minux]

Comment 1:

http://golang.org/cl/5905055/
cmd/go also doesn't like lib9, libbio, and libmach as cmd names.

Labels changed: added priority-go1, removed priority-triage.

Owner changed to @minux.

Status changed to Started.

[rsc]

Comment 2:

You cannot use 'cmd' as an import path,
just as you cannot use 'math'.  That's our name.
I don't believe the go command cares about lib9 etc though.
Russ

[alberts]

Comment 3:

Just to check, is it okay to use stuff under cmd? e.g. src/cmd/foo? Or is the whole
cmd/* reserved? If it is, an error might be better than having it mostly work.

[rsc]

Comment 4:

I believe cmd/* is reserved, just as math/* is reserved.
It does mostly work, and I don't want to change that,
but we are very clear everywhere we explain import paths
that if you want to avoid conflicts you need to use a
unique prefix.

[alberts]

Comment 5:

Okay, makes sense. Thanks. This is probably Retracted then.

[minux]

Comment 6:

cmd/go doesn't care about lib9, but the relevant code in go/build does
(http://golang.org/cl/5905055/).
You can try writing a command named lib9 and install it using 'go install -x ./...'.

[rsc]

Comment 7:

Thanks for the lib9 fix.

Labels changed: added priority-later, removed priority-go1.

[rsc]

Comment 8:

This issue was closed by revision 6718627.

Status changed to Fixed.