tests: tail-2/assert: reduce risk of race condition
authorPádraig Brady <P@draigBrady.com>
Tue, 8 Sep 2009 09:00:01 +0000 (10:00 +0100)
committerPádraig Brady <P@draigBrady.com>
Tue, 8 Sep 2009 09:01:45 +0000 (10:01 +0100)
* tests/tail2/assert: This reverts commit be853120, 25-08-2009,
"tests: tail-2/assert: avoid risk of race condition"
kill -0 doesn't send a signal and so will only confirm that the
background process was forked, which we know already because
we have its pid.

tests/tail-2/assert

index 623c368..416f206 100755 (executable)
@@ -42,13 +42,9 @@ tail_pid=$!
 
 echo sleeping for 7 seconds...
 
-# Wait for the backgrounded `tail' to start before removing foo.
+# Give the backgrounded `tail' a chance to start before removing foo.
 # Otherwise, without --retry, tail wouldn't try to open `foo' again.
-while :; do
-  env kill -0 $tail_pid && break
-  echo sleep .1
-  sleep .1
-done
+sleep 1
 
 rm -f foo
 sleep 6