]> git.ipfire.org Git - thirdparty/automake.git/commitdiff
test: Improve output of system information in test-suite.log.
authorCollin Funk <collin.funk1@gmail.com>
Mon, 17 Jun 2024 21:44:28 +0000 (14:44 -0700)
committerKarl Berry <karl@freefriends.org>
Mon, 17 Jun 2024 21:44:28 +0000 (14:44 -0700)
Fixes https://bugs.gnu.org/71421.

* lib/am/check.am ($(TEST_SUITE_LOG): Use $(AWK) instead of 'awk'.
Remove an extra space in uname output.  Start the line with
"System information", and other sections with "Distribution
information".
* NEWS: add reference to #71421.

NEWS
lib/am/check.am

diff --git a/NEWS b/NEWS
index e4699a5699985a66be1611cb55e1909e27053ffc..b0268d268c7c4e367c2c33b24735f92b96aa4a5d 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -46,7 +46,7 @@ New in 1.17:
 
   - test-suite.log now contains basic system information, and the
     console message about bug reporting on failure has a bit more detail.
-    (bug#68746)
+    (bug#68746, bug#71421)
 
   - When using the (default) "parallel" test driver, you can now omit the
     output of skipped tests from test-suite.log by defining the
index 526b445d557e73faba9b26eee95ed597effa3444..ab1411ac5140103896e531b5f484556873c2c1c5 100644 (file)
@@ -370,14 +370,26 @@ $(TEST_SUITE_LOG): $(TEST_LOGS)
 ## Function to write some basic system info in the log.
        output_system_information () \
        { \
-## Omit the hostname for privacy.  In practice it's a single word?
-          echo;                                    \
-         (uname -a | awk '{$$2=""; print}') 2>&1; \
-         if test -r /etc/os-release; then         \
-           sed 8q /etc/os-release;                \
-         elif test -r /etc/issue; then            \
-           cat /etc/issue;                        \
-         fi;                                      \
+## Omit the hostname for privacy.  Always a single word.
+## Start the line with a label for clarity, and so commands like
+## grep ^Linux don't get a false hit.
+          echo;                                     \
+         { uname -a | $(AWK) '{                    \
+  printf "System information (uname -a):";          \
+  for (i = 1; i < NF; ++i)                          \
+    {                                               \
+      if (i != 2)                                   \
+        printf " %s", $$i;                          \
+    }                                               \
+  printf "\n";                                      \
+}'; } 2>&1;                                         \
+         if test -r /etc/os-release; then          \
+           echo "Distribution information (/etc/os-release):"; \
+           sed 8q /etc/os-release;                 \
+         elif test -r /etc/issue; then             \
+           echo "Distribution information (/etc/issue):";      \
+           cat /etc/issue;                         \
+         fi;                                       \
        }; \
 ##
 ## Function to output the "please report bugs" message.