public inbox for gdb-cvs@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] [gdb/testsuite] Fix mi-sym-info.exp with check-read1
@ 2020-03-03  9:33 Tom de Vries
  0 siblings, 0 replies; only message in thread
From: Tom de Vries @ 2020-03-03  9:33 UTC (permalink / raw)
  To: gdb-cvs

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

commit b98cc2cf1aecd4b9348252832b4b0bef727866cf
Author: Tom de Vries <tdevries@suse.de>
Date:   Tue Mar 3 10:33:49 2020 +0100

    [gdb/testsuite] Fix mi-sym-info.exp with check-read1
    
    When running gdb.mi/mi-sym-info.exp with check-read1, we run into:
    ...
    FAIL: gdb.mi/mi-sym-info.exp: List all functions
    FAIL: gdb.mi/mi-sym-info.exp: List all variables
    ...
    
    The problem is that while the $mi_gdb_prompt is active, gdb_test_multiple is
    used without -prompt "$mi_gdb_prompt$", so it defaults to matching $gdb_prompt.
    
    Fix this by adding the missing gdb_test_multiple arguments.
    
    Reg-tested on x86_64-linux with make targets check and check-read1.
    
    gdb/testsuite/ChangeLog:
    
    2020-03-03  Tom de Vries  <tdevries@suse.de>
    
    	* gdb.mi/mi-sym-info.exp: Add missing -prompt "$mi_gdb_prompt$" to
    	gdb_test_multiple calls.

Diff:
---
 gdb/testsuite/ChangeLog              | 5 +++++
 gdb/testsuite/gdb.mi/mi-sym-info.exp | 6 ++++--
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index 55c0a3e..c3e154d 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2020-03-03  Tom de Vries  <tdevries@suse.de>
+
+	* gdb.mi/mi-sym-info.exp: Add missing -prompt "$mi_gdb_prompt$" to
+	gdb_test_multiple calls.
+
 2020-03-02  Andrew Burgess  <andrew.burgess@embecosm.com>
 
 	* gdb.base/default.exp: Remove test of 'set history filename'.
diff --git a/gdb/testsuite/gdb.mi/mi-sym-info.exp b/gdb/testsuite/gdb.mi/mi-sym-info.exp
index 9df683a..c07f3e8 100644
--- a/gdb/testsuite/gdb.mi/mi-sym-info.exp
+++ b/gdb/testsuite/gdb.mi/mi-sym-info.exp
@@ -68,7 +68,8 @@ mi_gdb_test "113-symbol-info-types" \
 # this command that we overflow expect's buffers, avoid this by
 # fetching the output piece by piece.
 set testname "List all functions"
-gdb_test_multiple "114-symbol-info-functions --include-nondebug" ${testname} {
+set cmd "114-symbol-info-functions --include-nondebug"
+gdb_test_multiple $cmd ${testname} -prompt "${mi_gdb_prompt}$" {
     -re "114\\^done,symbols=\{debug=\\\[${symtab_re}(?:,${symtab_re})*\\\],nondebug=\\\[" {
 	exp_continue
     }
@@ -83,7 +84,8 @@ gdb_test_multiple "114-symbol-info-functions --include-nondebug" ${testname} {
 }
 
 set testname "List all variables"
-gdb_test_multiple "115-symbol-info-variables --include-nondebug" ${testname} {
+set cmd "115-symbol-info-variables --include-nondebug"
+gdb_test_multiple $cmd ${testname} -prompt "${mi_gdb_prompt}$" {
     -re "115\\^done,symbols=\{debug=\\\[${symtab_re}(?:,${symtab_re})*\\\],nondebug=\\\[" {
 	verbose -log "Got the first part of the input"
 	exp_continue


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2020-03-03  9:33 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-03  9:33 [binutils-gdb] [gdb/testsuite] Fix mi-sym-info.exp with check-read1 Tom de Vries

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).