[PATCH 3/4] Cygwin: CI: Make stress test terser

Christian Franke Christian.Franke@t-online.de
Fri Apr 11 16:08:54 GMT 2025


Jon Turney wrote:
> Don't echo the command being run

OK, but this breaks -n option which is occasionally useful, see below.

> Capture stress-ng output to file
> Only show test output if it fails
>
> Capture all test output in an artifact

Are messages cygwin prints itself also captured? See below.


> ---
>   .github/workflows/cygwin.yml      | 10 ++++++++++
>   winsup/testsuite/stress/cygstress | 13 ++++++++-----
>   2 files changed, 18 insertions(+), 5 deletions(-)
>
> ...
>   
>   
> -  echo '$' "${cmd[@]}"
>     ! $dryrun || return 0

Possibly better:

if $dryrun; then
   echo '$' "${cmd[@]}"
   return 0
fi


>     (
> @@ -520,7 +523,7 @@ stress()
>   
>     mkdir "$td"
>     local rc=0
> -  "${cmd[@]}" || rc=$?
> +  "${cmd[@]}" >/dev/null || rc=$?

Redirect stderr to capture Cygwin's "panic" messages ?



More information about the Cygwin-patches mailing list