From 495eb3f922c8362949812fe65237d2158e25afe5 Mon Sep 17 00:00:00 2001 From: Tobias Klauser Date: Mon, 26 Feb 2018 11:15:41 +0100 Subject: syscall: remove/update outdated TODO comments Error returns for linux/arm syscalls are handled since a long time. Remove another list of unimplemented syscalls, following CL 96315. The root-only check in TestSyscallNoError was shown to be sufficient as part of CL 84485 already. NetBSD and OpenBSD do not implement the sendfile syscall (yet), so add a link to golang.org/issue/5847 Change-Id: I07efc3c3203537a4142707385f31b59dc0ecca42 Reviewed-on: https://go-review.googlesource.com/97115 Reviewed-by: Brad Fitzpatrick --- src/syscall/syscall_linux_test.go | 2 -- 1 file changed, 2 deletions(-) (limited to 'src/syscall/syscall_linux_test.go') diff --git a/src/syscall/syscall_linux_test.go b/src/syscall/syscall_linux_test.go index becd267101..932ccee491 100644 --- a/src/syscall/syscall_linux_test.go +++ b/src/syscall/syscall_linux_test.go @@ -180,8 +180,6 @@ func TestSyscallNoError(t *testing.T) { t.Skip("skipping on non-32bit architecture") } - // TODO(tklauser) is this check enough? Otherwise test for being in a non-k8s - // Linux VM via testenv.Builder(). if os.Getuid() != 0 { t.Skip("skipping root only test") } -- cgit v1.3-5-g9baa