go/build: reject Import("", ...)

[gopherbot]

by raul.san@sent.com:

build.Import is loading data about GOROOT instead of use GOPATH like I've set.

// * * *
package main

import (
    "fmt"
    "go/build"
    "log"
)

func main() {
    fmt.Println("GOROOT:", build.Default.GOROOT)
    fmt.Println("GOPATH:", build.Default.GOPATH)
    println()

    pkg, err := build.Import("", build.Default.GOPATH, build.FindOnly)
    if err != nil {
        log.Fatalf("GOROOT is not set: %s", err)
    }
    fmt.Println(pkg)
}
// * * *

GOROOT: /opt/go
GOPATH: /opt/go-3pkg:/home/under/go

&{/opt/go/src/pkg    /opt/go /opt/go/src/pkg /opt/go/pkg /opt/go/bin true
/opt/go/pkg/linux_amd64/.a [] [] [] [] [] [] [] [] [] [] map[] [] [] map[] [] [] map[]}

[rsc]

Comment 1:

Import paths found in GOROOT take precedence over import paths found in GOPATH.
You have asked for the empty import path, which should be rejected.
But since it is not rejected, and GOROOT+"/src/pkg/"+"" is a directory, Import uses that.

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

Status changed to Accepted.

[rsc]

Comment 2:

Labels changed: added go1.1.

[rsc]

Comment 3:

This issue was closed by revision ec9967f.

Status changed to Fixed.