]> Cypherpunks repositories - gostls13.git/commitdiff
os/user: use the stubs fallback for Android
authorElias Naur <elias.naur@gmail.com>
Sat, 4 Mar 2017 03:55:56 +0000 (04:55 +0100)
committerElias Naur <elias.naur@gmail.com>
Tue, 7 Mar 2017 19:38:24 +0000 (19:38 +0000)
Using the stubs, user.Current will no longer fail on Android, fixing
the os/exec.TestCredentialNoSetGroups test.

Change-Id: I8b9842aa6704c0cde383c549a614bab0a0ed7695
Reviewed-on: https://go-review.googlesource.com/37765
Run-TryBot: Elias Naur <elias.naur@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
src/os/user/lookup_android.go
src/os/user/lookup_stubs.go

index b1be3dc19310da0a8752eec05a6f433d44078c27..8ca30b8c274709181a498be767a8ae6d0a17bc1a 100644 (file)
@@ -8,15 +8,6 @@ package user
 
 import "errors"
 
-func init() {
-       userImplemented = false
-       groupImplemented = false
-}
-
-func current() (*User, error) {
-       return nil, errors.New("user: Current not implemented on android")
-}
-
 func lookupUser(string) (*User, error) {
        return nil, errors.New("user: Lookup not implemented on android")
 }
@@ -32,7 +23,3 @@ func lookupGroup(string) (*Group, error) {
 func lookupGroupId(string) (*Group, error) {
        return nil, errors.New("user: LookupGroupId not implemented on android")
 }
-
-func listGroups(*User) ([]string, error) {
-       return nil, errors.New("user: GroupIds not implemented on android")
-}
index f203c349befe82a92719b06d69811976983aec1c..7279617248cf9e33efa7831e3357cfc1a1c7c5a1 100644 (file)
@@ -2,7 +2,7 @@
 // Use of this source code is governed by a BSD-style
 // license that can be found in the LICENSE file.
 
-// +build !cgo,!windows,!plan9,!android
+// +build !cgo,!windows,!plan9 android
 
 package user
 
@@ -46,6 +46,9 @@ func current() (*User, error) {
 }
 
 func listGroups(*User) ([]string, error) {
+       if runtime.GOOS == "android" {
+               return nil, errors.New("user: GroupIds not implemented on Android")
+       }
        return nil, errors.New("user: GroupIds requires cgo")
 }