Commit 23ecad07 authored by Keith Randall's avatar Keith Randall

os/exec: tell lsof not to block

For some reason lsof is now hanging on my workstation
without the -b (avoid blocking in the kernel) option.
Adding -b makes the test pass and shouldn't hurt.

I don't know how recent the -b option is.  If the builders
are ok with it, it's probably ok.

LGTM=rsc
R=golang-codereviews, bradfitz, rsc
CC=golang-codereviews
https://golang.org/cl/166220043
parent 908dcab6
...@@ -246,7 +246,7 @@ func TestPipeLookPathLeak(t *testing.T) { ...@@ -246,7 +246,7 @@ func TestPipeLookPathLeak(t *testing.T) {
} }
func numOpenFDS(t *testing.T) (n int, lsof []byte) { func numOpenFDS(t *testing.T) (n int, lsof []byte) {
lsof, err := exec.Command("lsof", "-n", "-p", strconv.Itoa(os.Getpid())).Output() lsof, err := exec.Command("lsof", "-b", "-n", "-p", strconv.Itoa(os.Getpid())).Output()
if err != nil { if err != nil {
t.Skip("skipping test; error finding or running lsof") t.Skip("skipping test; error finding or running lsof")
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment