Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 25 Nov 2025 03:19:29 +0000
From:      Dag-Erling=?utf-8?Q? Sm=C3=B8rg?=rav <des@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Cc:        Bryan Drewery <bdrewery@FreeBSD.org>
Subject:   git: e9019e777ca3 - stable/14 - sh tests: Fix racy test11.0
Message-ID:  <69252041.33054.574763e9@gitrepo.freebsd.org>

index | next in thread | raw e-mail

The branch stable/14 has been updated by des:

URL: https://cgit.FreeBSD.org/src/commit/?id=e9019e777ca3bbb3217df7b54590dcb075430f4e

commit e9019e777ca3bbb3217df7b54590dcb075430f4e
Author:     Bryan Drewery <bdrewery@FreeBSD.org>
AuthorDate: 2025-11-07 17:28:37 +0000
Commit:     Dag-Erling Smørgrav <des@FreeBSD.org>
CommitDate: 2025-11-25 03:19:16 +0000

    sh tests: Fix racy test11.0
    
    This was sometimes exiting while the child fifo was created resulting in
    [ENOTEMPTY] from rm.  The child fifo isn't needed, just sleep.
    
    PR:     290837
    Fixes:  e31fb97148f ("read builtin: Empty variables on timeout")
    MFC after:      3 days
    
    (cherry picked from commit 5d55553e160a93788372785300f5bd07c2451f93)
---
 bin/sh/tests/builtins/read11.0 | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/bin/sh/tests/builtins/read11.0 b/bin/sh/tests/builtins/read11.0
index c75ed9c92a83..5bae80318b15 100644
--- a/bin/sh/tests/builtins/read11.0
+++ b/bin/sh/tests/builtins/read11.0
@@ -5,8 +5,8 @@ T=$(mktemp -d ${TMPDIR:-/tmp}/sh-test.XXXXXX)
 trap 'rm -rf "$T"' 0
 cd $T
 mkfifo fifo1
-# Open fifo1 for writing and then read block on a dummy fifo
-{ mkfifo fifo2; read dummy <fifo2; } >fifo1 &
+# Open fifo1 for writing
+{ sleep 10; } >fifo1 &
 # Wait for the child to open fifo1 for writing
 exec 3<fifo1
 v=original_value


help

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?69252041.33054.574763e9>