Re: RFA: make test suite transcripts

[Tom Tromey <tromey at redhat dot com>]

>>>>> "Eli" == Eli Zaretskii <eliz@gnu.org> writes:

>> The documentation needs a review.

Eli> Thanks.  The documentation patch is approved subject to the same
Eli> comments as for your previous patch to gdbint.texinfo.

Thanks.  BTW, this sort of detail work would be simpler if the manuals
were internally consistent.  I mostly copied existing texinfo usage
from elsewhere in gdbint.texinfo.

Anyway, later I decided that it is preferable to use a DejaGNU
variable, instead of an environment variable.  So, the text changed.
Let me know what you think.

Tom

2009-07-09  Tom Tromey  <tromey@redhat.com>

	* gdbint.texinfo (Testsuite): Document test transcripts.

2009-07-09  Tom Tromey  <tromey@redhat.com>

	* lib/gdb.exp: Handle LOG_GDB_SCRIPT.
	(remote_spawn, remote_close, send_gdb): New procs.

Index: doc/gdbint.texinfo
===================================================================
RCS file: /cvs/src/src/gdb/doc/gdbint.texinfo,v
retrieving revision 1.307
diff -u -r1.307 gdbint.texinfo
--- doc/gdbint.texinfo	2 Jul 2009 17:21:07 -0000	1.307
+++ doc/gdbint.texinfo	10 Jul 2009 17:22:10 -0000
@@ -7560,6 +7560,27 @@
 UNRESOLVED: gdb.base/example.exp: This test script does not work on a remote host.
 @end smallexample
 
+Sometimes it is convenient to get a transcript of the commands which
+the testsuite sends to @value{GDBN}.  For example, if @value{GDBN}
+crashes during testing, a transcript can be used to more easily
+reconstruct the failure when running @value{GDBN} under @value{GDBN}.
+
+You can instruct the @value{GDBN} testsuite to write transcripts by
+setting the DejaGNU variable @code{TRANSCRIPT} (to any value)
+before invoking @code{runtest} or @kbd{make check}.  The transcripts
+will be written into DejaGNU's output directory.  One transcript will
+be made for each invocation of @value{GDBN}; they will be named
+@file{transcript.@var{n}}, where @var{n} is an integer.  The first
+line of the transcript file will show how @value{GDBN} was invoked;
+each subsequent line is a command sent as input to @value{GDBN}.
+
+@smallexample
+make check RUNTESTFLAGS=TRANSCRIPT=y
+@end smallexample
+
+Note that the transcript is not always complete.  In particular, tests
+of completion can yield partial command lines.
+
 @section Testsuite Organization
 
 @cindex test suite organization
Index: testsuite/lib/gdb.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/lib/gdb.exp,v
retrieving revision 1.117
diff -u -r1.117 gdb.exp
--- testsuite/lib/gdb.exp	28 Jun 2009 00:20:24 -0000	1.117
+++ testsuite/lib/gdb.exp	10 Jul 2009 17:22:12 -0000
@@ -2983,3 +2983,49 @@
     return [get_integer_valueof "sizeof (${type})" $default]
 }
 
+# Log gdb command line and script if requested.
+if {[info exists TRANSCRIPT]} {
+  rename send_gdb real_send_gdb
+  rename remote_spawn real_remote_spawn
+  rename remote_close real_remote_close
+
+  global gdb_transcript
+  set gdb_transcript ""
+
+  global gdb_trans_count
+  set gdb_trans_count 1
+
+  proc remote_spawn {args} {
+    global gdb_transcript gdb_trans_count outdir
+
+    if {$gdb_transcript != ""} {
+      close $gdb_transcript
+    }
+    set gdb_transcript [open [file join $outdir transcript.$gdb_trans_count] w]
+    puts $gdb_transcript [lindex $args 1]
+    incr gdb_trans_count
+
+    return [uplevel real_remote_spawn $args]
+  }
+
+  proc remote_close {args} {
+    global gdb_transcript
+
+    if {$gdb_transcript != ""} {
+      close $gdb_transcript
+      set gdb_transcript ""
+    }
+
+    return [uplevel real_remote_close $args]
+  }
+
+  proc send_gdb {args} {
+    global gdb_transcript
+
+    if {$gdb_transcript != ""} {
+      puts -nonewline $gdb_transcript [lindex $args 0]
+    }
+
+    return [uplevel real_send_gdb $args]
+  }
+}