Collin Funk <[email protected]> writes:

>> I guess the reason for the test failure is the newer kernel
>> supporting disparate speeds on input and output,
>> causing the separated [io]speed to be displayed.
>> Note our tests run `stty 9600` so perhaps once the tests are run once,
>> you get the disparity on subsequent runs.
>
> Good guess. Upon starting my system today the tests fail. I see that a
> 'dnf update' on 10-31 updated my kernel:
>
>     $ dnf history info 149 | grep 'kernel-core' | awk '{ print $1 "\t" $2 }'
>     Install   kernel-core-0:6.17.5-300.fc43.x86_64
>     Remove    kernel-core-0:6.16.12-200.fc42.x86_64

I pushed the attached patch, since I could confirm it fixes the issue
on my system.

Collin

>From 170509d41ff3d586c13d0236b16d2ad639d276da Mon Sep 17 00:00:00 2001
Message-ID: <170509d41ff3d586c13d0236b16d2ad639d276da.1762110218.git.collin.fu...@gmail.com>
From: Collin Funk <[email protected]>
Date: Sun, 2 Nov 2025 10:57:21 -0800
Subject: [PATCH] tests: stty: filter out ispeed and ospeed from boolean
 options

* tests/stty/stty-pairs.sh: Also ignore lines starting with ispeed and
ospeed.
* tests/stty/stty.sh: Likewise.
Reported by Bernhard Voelker.
---
 tests/stty/stty-pairs.sh | 2 +-
 tests/stty/stty.sh       | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/stty/stty-pairs.sh b/tests/stty/stty-pairs.sh
index aebbd7fe7..2af9ac78c 100755
--- a/tests/stty/stty-pairs.sh
+++ b/tests/stty/stty-pairs.sh
@@ -38,7 +38,7 @@ stty $(cat $saved_state) || fail=1
 # Don't depend on terminal width.  Put each option on its own line,
 # remove all non-boolean ones, remove 'parenb' and 'cread' explicitly,
 # then remove any leading hyphens.
-sed_del='/^speed/d;/^rows/d;/^columns/d;/ = /d;s/parenb//;s/cread//'
+sed_del='/^[io]*speed/d;/^rows/d;/^columns/d;/ = /d;s/parenb//;s/cread//'
 options=$(stty -a | tr -s ';' '\n' | sed "s/^ //;$sed_del;s/-//g")
 
 # Take them in pairs, with and without the leading '-'.
diff --git a/tests/stty/stty.sh b/tests/stty/stty.sh
index c0f74947a..b224ae41f 100755
--- a/tests/stty/stty.sh
+++ b/tests/stty/stty.sh
@@ -51,7 +51,7 @@ returns_ 1 stty -raw -a 2>/dev/null || fail=1
 # Build a list of all boolean options stty accepts on this system.
 # Don't depend on terminal width.  Put each option on its own line,
 # remove all non-boolean ones, then remove any leading hyphens.
-sed_del='/^speed/d;/^rows/d;/^columns/d;/ = /d'
+sed_del='/^[io]*speed/d;/^rows/d;/^columns/d;/ = /d'
 options=$(stty -a | tr -s ';' '\n' | sed "s/^ //;$sed_del;s/-//g")
 
 # Take them one at a time, with and without the leading '-'.
-- 
2.51.1

Reply via email to