]> Cypherpunks repositories - gostls13.git/commitdiff
os/exec: re-enable TestExtraFiles for plan9
authorRichard Miller <miller.research@gmail.com>
Fri, 29 Apr 2016 20:02:59 +0000 (21:02 +0100)
committerDavid du Colombier <0intro@gmail.com>
Thu, 5 May 2016 22:01:06 +0000 (22:01 +0000)
This test should now succeed after CL 22610 which fixes issue #7118

Change-Id: Ie785a84d77b27c832a1ddd81699bf25dab24b97d
Reviewed-on: https://go-review.googlesource.com/22640
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: David du Colombier <0intro@gmail.com>
Run-TryBot: David du Colombier <0intro@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>

src/os/exec/exec_test.go

index 1151ca7d0f0c26dd7d34ce698ff7e3cd614fb5b9..0cff3bb9264e7f41fc88f3fd89b1cb193f154660 100644 (file)
@@ -660,10 +660,6 @@ func TestHelperProcess(*testing.T) {
                        // the cloned file descriptors that result from opening
                        // /dev/urandom.
                        // https://golang.org/issue/3955
-               case "plan9":
-                       // TODO(0intro): Determine why Plan 9 is leaking
-                       // file descriptors.
-                       // https://golang.org/issue/7118
                case "solaris":
                        // TODO(aram): This fails on Solaris because libc opens
                        // its own files, as it sees fit. Darwin does the same,