From be8531206b0554bdac6cfb8809c4a2dab56b5a52 Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Tue, 25 Aug 2009 20:43:13 +0200 Subject: tests: tail-2/assert: avoid risk of race condition * tests/tail-2/assert: Avoid spurious failure due to race condition. Rather than sleeping for 1 second and crossing fingers, wait explicitly for backgrounded tail process to start. Otherwise, this test would fail under heavy load. --- tests/tail-2/assert | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'tests') diff --git a/tests/tail-2/assert b/tests/tail-2/assert index 416f206a1..623c3684f 100755 --- a/tests/tail-2/assert +++ b/tests/tail-2/assert @@ -42,9 +42,13 @@ tail_pid=$! echo sleeping for 7 seconds... -# Give the backgrounded `tail' a chance to start before removing foo. +# Wait for the backgrounded `tail' to start before removing foo. # Otherwise, without --retry, tail wouldn't try to open `foo' again. -sleep 1 +while :; do + env kill -0 $tail_pid && break + echo sleep .1 + sleep .1 +done rm -f foo sleep 6 -- cgit v1.2.3-54-g00ecf