diff options
| -rw-r--r-- | ChangeLog | 14 | ||||
| -rw-r--r-- | commands/report-card.awk | 3 | ||||
| -rw-r--r-- | doc/dejagnu.texi | 2 |
3 files changed, 17 insertions, 2 deletions
@@ -1,3 +1,17 @@ +2024-10-20 Jacob Bachmeyer <jcb@gnu.org> + + PR73907 + + * doc/dejagnu.texi (Writing a test case): Fix typo reported by + Heiko Eissfeldt. + +2024-09-23 Jacob Bachmeyer <jcb@gnu.org> + + * commands/report-card.awk: When preparing to compute totals and + setting $0 to the list of possible test results, explicitly set NF + to 9 to work around a bug in AIX 7.1 awk, which fails to update NF + when $0 is assigned from a constant string. + 2024-09-18 Jacob Bachmeyer <jcb@gnu.org> * dejagnu: Adapt Zsh compatibility prologue from configure. diff --git a/commands/report-card.awk b/commands/report-card.awk index 415e74a..a88b5cb 100644 --- a/commands/report-card.awk +++ b/commands/report-card.awk @@ -1,5 +1,5 @@ # report-card.awk -- Test summary tool -# Copyright (C) 2018, 2021 Free Software Foundation, Inc. +# Copyright (C) 2018, 2021, 2024 Free Software Foundation, Inc. # # This file is part of DejaGnu. # @@ -132,6 +132,7 @@ $1 ~ /:$/ { sub(/:$/, "", $1); Totals["tp", Tool, Pass, $1]++ } END { $0 = ("PASS FAIL KPASS KFAIL XPASS XFAIL UNSUPPORTED UNRESOLVED UNTESTED") + NF = 9 # work around bug in AIX 7.1 awk for (i = 1; i in Tools; i++) for (j = 1; ("t", Tools[i], j) in Passes; j++) for (k = 1; k <= NF; k++) { diff --git a/doc/dejagnu.texi b/doc/dejagnu.texi index 201f33d..61739a7 100644 --- a/doc/dejagnu.texi +++ b/doc/dejagnu.texi @@ -2190,7 +2190,7 @@ built up a family of Tcl procedures specialized for GDB testing. @cindex hints on writing a test case @cindex test cases, writing -To preserve basic sanity, no should test ever pass if there was any +To preserve basic sanity, no test should ever pass if there was any kind of problem in the test case. To take an extreme case, tests that pass even when the tool will not spawn are misleading. Ideally, a test in this sort of situation should not fail either. Instead, print |