Commit 1a14b5ba authored by Brad Fitzpatrick's avatar Brad Fitzpatrick

cmd/api: don't depend on os/user or USER to check api

The -nocgo builder failed because it has cgo disabled
as well as no USER environment variable:
http://build.golang.org/log/2250abb82f5022b72a12997b8ff89fcdeff094c9

# Checking API compatibility.
Error getting current user: user: Current not implemented on linux/amd64
exit status 1

Don't require the environment variable here.

LGTM=minux
R=dave, adg, minux
CC=golang-codereviews
https://golang.org/cl/140290043
parent a2943710
......@@ -98,11 +98,9 @@ func prepGoPath() string {
if err == nil {
username = u.Username
} else {
// Only need to handle Unix here, as Windows's os/user uses
// native syscall and should work fine without cgo.
username = os.Getenv("USER")
if username == "" {
log.Fatalf("Error getting current user: %v", err)
username = "nobody"
}
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment