[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