From 1a14b5bad8926941023ca310fa3b53c70717b1e2 Mon Sep 17 00:00:00 2001 From: Brad Fitzpatrick Date: Thu, 4 Sep 2014 17:13:22 -0700 Subject: [PATCH] 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 --- src/cmd/api/run.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/cmd/api/run.go b/src/cmd/api/run.go index 896b2b4a1a..c323deb603 100644 --- a/src/cmd/api/run.go +++ b/src/cmd/api/run.go @@ -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" } } -- 2.50.0