[binutils-gdb] Print command trace to gdb_stdlog

[Tom Tromey]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=e4d214db01492b035874eaac2c3a022e6c3d5bc8

commit e4d214db01492b035874eaac2c3a022e6c3d5bc8
Author: Tom Tromey <tromey@adacore.com>
Date:   Tue Jun 2 11:27:15 2026 -0600

    Print command trace to gdb_stdlog
    
    I recently wanted to enable some gdb logging when running the internal
    AdaCore test suite.  To do this nicely, I enable debug-redirect early
    in the test, so that the logging output does not affect the test
    results.
    
    I also wanted to the log the commands, to correlate what I see in the
    debug log with what I see in the ordinary test suite log file (which
    is basically like our own gdb.log).
    
    However, I found that "set trace-commands on" will log to stdout, not
    stdlog.  This patch changes this to log to the log file instead.

Diff:
---
 gdb/cli/cli-script.c                   | 10 ++++------
 gdb/testsuite/gdb.base/ui-redirect.exp |  6 ++++++
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/gdb/cli/cli-script.c b/gdb/cli/cli-script.c
index b8312f431cb..0a1208845b4 100644
--- a/gdb/cli/cli-script.c
+++ b/gdb/cli/cli-script.c
@@ -486,8 +486,6 @@ ATTRIBUTE_PRINTF (1, 2)
 void
 print_command_trace (const char *fmt, ...)
 {
-  int i;
-
   if (suppress_next_print_command_trace)
     {
       suppress_next_print_command_trace = false;
@@ -497,15 +495,15 @@ print_command_trace (const char *fmt, ...)
   if (!source_verbose && !trace_commands)
     return;
 
-  for (i=0; i < command_nest_depth; i++)
-    gdb_printf ("+");
+  for (int i = 0; i < command_nest_depth; ++i)
+    gdb_printf (gdb_stdlog, "+");
 
   va_list args;
 
   va_start (args, fmt);
-  gdb_vprintf (fmt, args);
+  gdb_vprintf (gdb_stdlog, fmt, args);
   va_end (args);
-  gdb_puts ("\n");
+  gdb_puts ("\n", gdb_stdlog);
 }
 
 /* Helper for execute_control_command.  */
diff --git a/gdb/testsuite/gdb.base/ui-redirect.exp b/gdb/testsuite/gdb.base/ui-redirect.exp
index 313843e3626..bed11bd2e87 100644
--- a/gdb/testsuite/gdb.base/ui-redirect.exp
+++ b/gdb/testsuite/gdb.base/ui-redirect.exp
@@ -135,6 +135,12 @@ with_test_prefix "redirect debugging" {
     "Copying output to /dev/null.*Redirecting debug output to /dev/null\\."
     gdb_test "continue" "Continuing.*((?!infrun).).*Breakpoint ${::decimal}, bar.*"
     gdb_test "set debug infrun 0"
+
+    # The trace should be sent to the log, not stdout.
+    gdb_test_no_output "set trace-commands on"
+    gdb_test_no_output {printf ""}
+    gdb_test_no_output "set trace-commands off"
+
     gdb_test "set logging enabled off" "Done logging to /dev/null\\."
     gdb_test "help" "List of classes of commands:.*"
 }