]> Cypherpunks repositories - gostls13.git/commitdiff
os/exec: fix number of expected file descriptors on Plan 9
authorDavid du Colombier <0intro@gmail.com>
Mon, 20 Oct 2014 21:01:32 +0000 (23:01 +0200)
committerDavid du Colombier <0intro@gmail.com>
Mon, 20 Oct 2014 21:01:32 +0000 (23:01 +0200)
Since CL 104570043 and 112720043, we are using the
nsec system call instead of /dev/bintime on Plan 9.

LGTM=rsc
R=rsc
CC=aram, golang-codereviews
https://golang.org/cl/155590043

src/os/exec/exec_test.go

index 5fd439b8bb297c5e0b5fd463ea409c07d4e60a18..bc9c00effe554db124c15cde971dc93a02281bb4 100644 (file)
@@ -258,15 +258,7 @@ var testedAlreadyLeaked = false
 // basefds returns the number of expected file descriptors
 // to be present in a process at start.
 func basefds() uintptr {
-       n := os.Stderr.Fd() + 1
-
-       // Go runtime for 32-bit Plan 9 requires that /dev/bintime
-       // be kept open.
-       // See ../../runtime/time_plan9_386.c:/^runtime·nanotime
-       if runtime.GOOS == "plan9" && runtime.GOARCH == "386" {
-               n++
-       }
-       return n
+       return os.Stderr.Fd() + 1
 }
 
 func closeUnexpectedFds(t *testing.T, m string) {