public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [pushed 0/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host/target
@ 2023-03-22  8:38 Tom de Vries
  2023-03-22  8:38 ` [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host Tom de Vries
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: Tom de Vries @ 2023-03-22  8:38 UTC (permalink / raw)
  To: gdb-patches

Fix a few test-cases in gdb.cp/*.exp for various remote host/target boards.

Results:
...
 $ ./test-all.sh
 HOST/TARGET: local
 # of expected failures          14
 # of expected passes            8576
 # of known failures             21
 HOST: local, TARGET: native-gdbserver
 # of expected failures          14
 # of expected passes            8552
 # of known failures             21
 # of unsupported tests          2
 HOST: local, TARGET: native-extended-gdbserver
 # of expected failures          14
 # of expected passes            8576
 # of known failures             21
 HOST: local, TARGET: native-stdio-gdbserver
 # of expected failures          14
 # of expected passes            8552
 # of known failures             21
 # of unsupported tests          2
 HOST: local, TARGET: remote-gdbserver-on-localhost
 # of expected failures          14
 # of expected passes            8560
 # of known failures             21
 # of unsupported tests          2
 HOST: local, TARGET: remote-stdio-gdbserver
 # of expected failures          14
 # of expected passes            8560
 # of known failures             21
 # of unsupported tests          2
 HOST: local, TARGET: remote-stdio-gdbserver-remotedir
 # of expected failures          14
 # of expected passes            8560
 # of known failures             21
 # of unsupported tests          2
 HOST: local-remote-host-notty, TARGET: native-gdbserver
 # of expected failures          14
 # of expected passes            8540
 # of known failures             21
 # of unsupported tests          2
 # of untested testcases         1
 HOST: local-remote-host-notty, TARGET: native-extended-gdbserver
 # of expected failures          14
 # of expected passes            8564
 # of known failures             21
 # of untested testcases         1
 HOST: local-remote-host-notty, TARGET: remote-gdbserver-on-localhost
 # of expected failures          14
 # of expected passes            8540
 # of known failures             21
 # of unsupported tests          2
 # of untested testcases         1
 HOST: local-remote-host-notty, TARGET: remote-stdio-gdbserver
 # of expected failures          14
 # of expected passes            8540
 # of known failures             21
 # of unsupported tests          2
 # of untested testcases         1
 HOST: local-remote-host-notty, TARGET: remote-stdio-gdbserver-remotedir
 # of expected failures          14
 # of expected passes            8540
 # of known failures             21
 # of unsupported tests          2
 # of untested testcases         1
 HOST: local-remote-host, TARGET: native-gdbserver
 # of expected failures          14
 # of expected passes            8560
 # of known failures             21
 # of unsupported tests          2
 HOST: local-remote-host, TARGET: native-extended-gdbserver
 # of expected failures          14
 # of expected passes            8584
 # of known failures             21
 HOST: local-remote-host, TARGET: remote-gdbserver-on-localhost
 # of expected failures          14
 # of expected passes            8560
 # of known failures             21
 # of unsupported tests          2
 HOST: local-remote-host, TARGET: remote-stdio-gdbserver
 # of expected failures          14
 # of expected passes            8560
 # of known failures             21
 # of unsupported tests          2
 HOST: local-remote-host, TARGET: remote-stdio-gdbserver-remotedir
 # of expected failures          14
 # of expected passes            8560
 # of known failures             21
 # of unsupported tests          2
 HOST/TARGET: local-remote-host-native
 # of expected failures          14
 # of expected passes            8584
 # of known failures             21
...

Tom de Vries (3):
  [gdb/testsuite] Fix gdb.cp/*.exp for remote host
  [gdb/testsuite] Fix gdb.cp/annota{2,3}.exp for
    native-extended-gdbserver
  [gdb/testsuite] Fix gdb.cp/cp-relocate.exp for remote host

 gdb/testsuite/gdb.cp/annota2.exp         | 15 ++++++++++++--
 gdb/testsuite/gdb.cp/annota3.exp         | 25 ++++++++++++++++--------
 gdb/testsuite/gdb.cp/breakpoint-locs.exp |  2 ++
 gdb/testsuite/gdb.cp/cp-relocate.exp     |  4 +++-
 gdb/testsuite/gdb.cp/filename.exp        |  2 ++
 gdb/testsuite/gdb.cp/gdb2384.exp         |  2 ++
 gdb/testsuite/gdb.cp/hang.exp            |  2 ++
 gdb/testsuite/gdb.cp/m-static.exp        |  8 +++++++-
 gdb/testsuite/gdb.cp/mb-inline.exp       |  1 +
 gdb/testsuite/gdb.cp/minsym-fallback.exp |  3 +++
 gdb/testsuite/gdb.cp/ovsrch.exp          |  2 ++
 gdb/testsuite/gdb.cp/pr10728.exp         |  2 ++
 gdb/testsuite/gdb.cp/rtti.exp            |  2 ++
 gdb/testsuite/gdb.cp/stub-array-size.exp |  2 ++
 gdb/testsuite/gdb.cp/subtypes.exp        |  2 ++
 gdb/testsuite/lib/gdb.exp                | 12 ++++++++++++
 16 files changed, 74 insertions(+), 12 deletions(-)


base-commit: 100f993c53a50352fd91554927142a2558f3481c
-- 
2.35.3


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host
  2023-03-22  8:38 [pushed 0/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host/target Tom de Vries
@ 2023-03-22  8:38 ` Tom de Vries
  2023-03-22 15:05   ` Simon Marchi
  2023-03-22  8:38 ` [pushed 2/3] [gdb/testsuite] Fix gdb.cp/annota{2,3}.exp for native-extended-gdbserver Tom de Vries
  2023-03-22  8:38 ` [pushed 3/3] [gdb/testsuite] Fix gdb.cp/cp-relocate.exp for remote host Tom de Vries
  2 siblings, 1 reply; 7+ messages in thread
From: Tom de Vries @ 2023-03-22  8:38 UTC (permalink / raw)
  To: gdb-patches

Fix a few test-cases in gdb.cp/*.exp for remote host using new proc
include_file.

Tested on x86_64-linux.
---
 gdb/testsuite/gdb.cp/breakpoint-locs.exp |  2 ++
 gdb/testsuite/gdb.cp/filename.exp        |  2 ++
 gdb/testsuite/gdb.cp/gdb2384.exp         |  2 ++
 gdb/testsuite/gdb.cp/hang.exp            |  2 ++
 gdb/testsuite/gdb.cp/m-static.exp        |  8 +++++++-
 gdb/testsuite/gdb.cp/mb-inline.exp       |  1 +
 gdb/testsuite/gdb.cp/minsym-fallback.exp |  3 +++
 gdb/testsuite/gdb.cp/ovsrch.exp          |  2 ++
 gdb/testsuite/gdb.cp/pr10728.exp         |  2 ++
 gdb/testsuite/gdb.cp/rtti.exp            |  2 ++
 gdb/testsuite/gdb.cp/stub-array-size.exp |  2 ++
 gdb/testsuite/gdb.cp/subtypes.exp        |  2 ++
 gdb/testsuite/lib/gdb.exp                | 12 ++++++++++++
 13 files changed, 41 insertions(+), 1 deletion(-)

diff --git a/gdb/testsuite/gdb.cp/breakpoint-locs.exp b/gdb/testsuite/gdb.cp/breakpoint-locs.exp
index 0d1ad68f9f5..11192654a35 100644
--- a/gdb/testsuite/gdb.cp/breakpoint-locs.exp
+++ b/gdb/testsuite/gdb.cp/breakpoint-locs.exp
@@ -19,6 +19,8 @@ require allow_cplus_tests
 
 standard_testfile .cc breakpoint-locs-2.cc
 
+include_file breakpoint-locs.h
+
 if { [prepare_for_testing "failed to prepare" $testfile "$srcfile $srcfile2"\
 	  {debug c++}] } {
     return -1
diff --git a/gdb/testsuite/gdb.cp/filename.exp b/gdb/testsuite/gdb.cp/filename.exp
index 9b1d2b8a606..2eddb9b7982 100644
--- a/gdb/testsuite/gdb.cp/filename.exp
+++ b/gdb/testsuite/gdb.cp/filename.exp
@@ -17,6 +17,8 @@ require allow_cplus_tests
 
 standard_testfile .cc
 
+include_file includefile
+
 if {[prepare_for_testing "failed to prepare" $testfile $srcfile {debug c++}]} {
     return -1
 }
diff --git a/gdb/testsuite/gdb.cp/gdb2384.exp b/gdb/testsuite/gdb.cp/gdb2384.exp
index 46d93d27e7b..7c6b27fd8db 100644
--- a/gdb/testsuite/gdb.cp/gdb2384.exp
+++ b/gdb/testsuite/gdb.cp/gdb2384.exp
@@ -25,6 +25,8 @@ require allow_cplus_tests allow_shlib_tests
 
 standard_testfile .cc gdb2384-base.cc
 
+include_file gdb2384-base.h
+
 set sofile [standard_output_file gdb2384-base.so]
 
 if { [gdb_compile_shlib $srcdir/$subdir/$srcfile2 $sofile {debug c++}] != ""
diff --git a/gdb/testsuite/gdb.cp/hang.exp b/gdb/testsuite/gdb.cp/hang.exp
index 7d8588520ec..71ab5c7a63b 100644
--- a/gdb/testsuite/gdb.cp/hang.exp
+++ b/gdb/testsuite/gdb.cp/hang.exp
@@ -18,6 +18,8 @@ require allow_cplus_tests
 
 standard_testfile hang1.cc hang2.cc hang3.cc
 
+include_file hang.H
+
 if {[prepare_for_testing "failed to prepare" $testfile \
 	 [list $srcfile $srcfile2 $srcfile3] {debug c++}]} {
     return -1
diff --git a/gdb/testsuite/gdb.cp/m-static.exp b/gdb/testsuite/gdb.cp/m-static.exp
index 56c4cdad95e..419d772e28e 100644
--- a/gdb/testsuite/gdb.cp/m-static.exp
+++ b/gdb/testsuite/gdb.cp/m-static.exp
@@ -27,6 +27,8 @@ require allow_cplus_tests
 
 standard_testfile .cc m-static1.cc
 
+include_file m-static.h
+
 set flags [list debug c++]
 if { [test_compiler_info gcc-*] && [gcc_major_version] >= 10 } {
     # Work around PR gcc/101452.
@@ -181,7 +183,11 @@ gdb_test "print test4.somewhere" "\\$\[0-9\].* = 3.14\[0-9\]*" "static const flo
 
 # Also make sure static const members can be found via "info var".
 if { $non_dwarf } { setup_xfail *-*-* }
-gdb_test "info variable everywhere" "File .*/m-static\[.\]h.*const int gnu_obj_4::everywhere;"
+gdb_test "info variable everywhere" \
+    [multi_line \
+	 "File (.*/)?m-static\[.\]h:" \
+	 "$decimal:\tconst int gnu_obj_4::everywhere;" \
+	 ".*"]
 
 # Perhaps at some point test4 should also include a test for a static
 # const int that was initialized in the header file.  But I'm not sure
diff --git a/gdb/testsuite/gdb.cp/mb-inline.exp b/gdb/testsuite/gdb.cp/mb-inline.exp
index 555827806f7..9793b14c073 100644
--- a/gdb/testsuite/gdb.cp/mb-inline.exp
+++ b/gdb/testsuite/gdb.cp/mb-inline.exp
@@ -22,6 +22,7 @@ require allow_cplus_tests
 
 standard_testfile mb-inline1.cc mb-inline2.cc
 set hdrfile "${testfile}.h"
+include_file $hdrfile
 
 if {[prepare_for_testing "failed to prepare" $testfile \
 	 [list $srcfile $srcfile2] {debug c++}]} {
diff --git a/gdb/testsuite/gdb.cp/minsym-fallback.exp b/gdb/testsuite/gdb.cp/minsym-fallback.exp
index 56a56a3da6b..a3bfe56bec6 100644
--- a/gdb/testsuite/gdb.cp/minsym-fallback.exp
+++ b/gdb/testsuite/gdb.cp/minsym-fallback.exp
@@ -18,6 +18,9 @@
 # a different file providing only ELF symbols.
 
 standard_testfile .cc minsym-fallback-main.cc
+
+include_file minsym-fallback.h
+
 set executable $testfile
 set objfile [standard_output_file ${testfile}.o]
 set objmainfile [standard_output_file ${testfile}-main.o]
diff --git a/gdb/testsuite/gdb.cp/ovsrch.exp b/gdb/testsuite/gdb.cp/ovsrch.exp
index 31a8abf7793..39e022c6676 100644
--- a/gdb/testsuite/gdb.cp/ovsrch.exp
+++ b/gdb/testsuite/gdb.cp/ovsrch.exp
@@ -57,6 +57,8 @@ require allow_cplus_tests
 # Test for c++/11734
 standard_testfile ovsrch1.cc ovsrch2.cc ovsrch3.cc ovsrch4.cc
 
+include_file ovsrch.h
+
 if {[prepare_for_testing "failed to prepare" $testfile \
 	 [list $srcfile $srcfile2 $srcfile3 $srcfile4] {c++ debug}]} {
     return -1
diff --git a/gdb/testsuite/gdb.cp/pr10728.exp b/gdb/testsuite/gdb.cp/pr10728.exp
index b860fb7c3ae..ea9c1242725 100644
--- a/gdb/testsuite/gdb.cp/pr10728.exp
+++ b/gdb/testsuite/gdb.cp/pr10728.exp
@@ -22,6 +22,8 @@ require allow_cplus_tests
 load_lib "cp-support.exp"
 
 standard_testfile pr10728-x.cc pr10728-y.cc
+include_file pr10728-x.h
+
 set tfx [standard_output_file pr10728-x.o]
 set tfy [standard_output_file pr10728-y.o]
 
diff --git a/gdb/testsuite/gdb.cp/rtti.exp b/gdb/testsuite/gdb.cp/rtti.exp
index 8ad7683ee40..18dfaf0c2bd 100644
--- a/gdb/testsuite/gdb.cp/rtti.exp
+++ b/gdb/testsuite/gdb.cp/rtti.exp
@@ -34,6 +34,8 @@ require allow_cplus_tests
 
 standard_testfile rtti1.cc rtti2.cc
 
+include_file rtti.h
+
 if {[prepare_for_testing "failed to prepare" $testfile \
 	 [list $srcfile $srcfile2] {debug c++}]} {
     return -1
diff --git a/gdb/testsuite/gdb.cp/stub-array-size.exp b/gdb/testsuite/gdb.cp/stub-array-size.exp
index 0afe232d583..7757db20af9 100644
--- a/gdb/testsuite/gdb.cp/stub-array-size.exp
+++ b/gdb/testsuite/gdb.cp/stub-array-size.exp
@@ -22,6 +22,8 @@ require allow_cplus_tests
 
 standard_testfile .cc stub-array-size2.cc
 
+include_file stub-array-size.h
+
 if {[prepare_for_testing "failed to prepare" $testfile "$srcfile $srcfile2" \
 	{c++ debug}]} {
     return -1
diff --git a/gdb/testsuite/gdb.cp/subtypes.exp b/gdb/testsuite/gdb.cp/subtypes.exp
index f5cf349cc26..afad478cba3 100644
--- a/gdb/testsuite/gdb.cp/subtypes.exp
+++ b/gdb/testsuite/gdb.cp/subtypes.exp
@@ -22,6 +22,8 @@ load_lib "cp-support.exp"
 
 standard_testfile .cc subtypes-2.cc
 
+include_file subtypes.h
+
 if {[prepare_for_testing "failed to prepare" $testfile \
 	 [list $srcfile $srcfile2] \
 	 {debug c++ additional_flags=-std=c++11}]} {
diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp
index 0a0ae697d1e..24fcb8f1746 100644
--- a/gdb/testsuite/lib/gdb.exp
+++ b/gdb/testsuite/lib/gdb.exp
@@ -9554,6 +9554,18 @@ gdb_caching_proc have_compile_flag { flag } {
 		additional_flags=$flag]
 }
 
+# Handle include file $srcdir/$subdir/FILE.
+
+proc include_file { file } {
+    set file [file join $::srcdir $::subdir $file]
+    if { [is_remote host] } {
+	set res [remote_download host $file]
+    } else {
+	set res $file
+    }
+
+    return $res
+}
 
 # Handle include file FILE, and if necessary update compiler flags variable
 # FLAGS.
-- 
2.35.3


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [pushed 2/3] [gdb/testsuite] Fix gdb.cp/annota{2,3}.exp for native-extended-gdbserver
  2023-03-22  8:38 [pushed 0/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host/target Tom de Vries
  2023-03-22  8:38 ` [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host Tom de Vries
@ 2023-03-22  8:38 ` Tom de Vries
  2023-03-22  8:38 ` [pushed 3/3] [gdb/testsuite] Fix gdb.cp/cp-relocate.exp for remote host Tom de Vries
  2 siblings, 0 replies; 7+ messages in thread
From: Tom de Vries @ 2023-03-22  8:38 UTC (permalink / raw)
  To: gdb-patches

When running test-cases gdb.cp/annota{2,3}.exp with target board
native-extended-gdbserver, we run into a few FAILs, due to the test-cases
trying to match inferior output together with gdb output.

Fix this by ignoring the inferior output in this case.

Tested on x86_64-linux.
---
 gdb/testsuite/gdb.cp/annota2.exp | 15 +++++++++++++--
 gdb/testsuite/gdb.cp/annota3.exp | 25 +++++++++++++++++--------
 2 files changed, 30 insertions(+), 10 deletions(-)

diff --git a/gdb/testsuite/gdb.cp/annota2.exp b/gdb/testsuite/gdb.cp/annota2.exp
index 64adbbdb220..506817bd498 100644
--- a/gdb/testsuite/gdb.cp/annota2.exp
+++ b/gdb/testsuite/gdb.cp/annota2.exp
@@ -126,13 +126,24 @@ gdb_test_multiple "print a" "print class" {
 # `a.x is 1' is asynchronous regarding to `frames-invalid'.
 #
 
+if { $gdb_spawn_id == $inferior_spawn_id } {
+    set match_output \
+	[multi_line \
+	     "\($frames_invalid\)*a.x is 1" \
+	     "\($frames_invalid\)*"]
+} else {
+    # Usually we'd handle this using gdb_test_stdio, but that looks too
+    # complicated in combination with annotations, so just ignore the inferior
+    # output.
+    set match_output "\($frames_invalid\)*"
+}
+
 set pat [multi_line "" \
 	     "\032\032post-prompt" \
 	     "Continuing." \
 	     "" \
 	     "\032\032starting" \
-	     "\(${frames_invalid}\)*a.x is 1" \
-	     "\(${frames_invalid}\)*"\
+	     $match_output \
 	     "\032\032exited 0" \
 	     "$inferior_exited_re normally." \
 	     "" \
diff --git a/gdb/testsuite/gdb.cp/annota3.exp b/gdb/testsuite/gdb.cp/annota3.exp
index 788bda77f3d..7b85c84cd97 100644
--- a/gdb/testsuite/gdb.cp/annota3.exp
+++ b/gdb/testsuite/gdb.cp/annota3.exp
@@ -95,14 +95,23 @@ gdb_expect_list "print class" "$gdb_prompt$" {
 # annotate-exited
 #
 send_gdb "continue\n"
-gdb_expect_list "continue to exit" "$gdb_prompt$" [concat {
-    "\r\n\032\032post-prompt\r\n"
-    "Continuing.\r\n"
-    "\r\n\032\032starting\r\n"
-    "a.x is 1\r\n"
-    "\r\n\032\032exited 0\r\n"} [list "$inferior_exited_re normally.\r\n"] {
-    "\r\n\032\032stopped\r\n"
-}]
+
+set el {}
+lappend el "\r\n\032\032post-prompt\r\n"
+lappend el "Continuing.\r\n"
+lappend el "\r\n\032\032starting\r\n"
+if { $gdb_spawn_id == $inferior_spawn_id } {
+    lappend el "a.x is 1\r\n"
+} else {
+    # Usually we'd handle this using gdb_test_stdio, but that looks too
+    # complicated in combination with annotations, so just ignore the inferior
+    # output.
+}
+lappend el "\r\n\032\032exited 0\r\n"
+lappend el "$inferior_exited_re normally.\r\n"
+lappend el "\r\n\032\032stopped\r\n"
+
+gdb_expect_list "continue to exit" "$gdb_prompt$" $el
 
 #
 # delete all breakpoints
-- 
2.35.3


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [pushed 3/3] [gdb/testsuite] Fix gdb.cp/cp-relocate.exp for remote host
  2023-03-22  8:38 [pushed 0/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host/target Tom de Vries
  2023-03-22  8:38 ` [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host Tom de Vries
  2023-03-22  8:38 ` [pushed 2/3] [gdb/testsuite] Fix gdb.cp/annota{2,3}.exp for native-extended-gdbserver Tom de Vries
@ 2023-03-22  8:38 ` Tom de Vries
  2 siblings, 0 replies; 7+ messages in thread
From: Tom de Vries @ 2023-03-22  8:38 UTC (permalink / raw)
  To: gdb-patches

Fix test-case gdb.cp/cp-relocate.exp for remote host using
gdb_remote_download.

Tested on x86_64-linux.
---
 gdb/testsuite/gdb.cp/cp-relocate.exp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/gdb/testsuite/gdb.cp/cp-relocate.exp b/gdb/testsuite/gdb.cp/cp-relocate.exp
index d400bb14e46..25ed15a9aa1 100644
--- a/gdb/testsuite/gdb.cp/cp-relocate.exp
+++ b/gdb/testsuite/gdb.cp/cp-relocate.exp
@@ -118,7 +118,9 @@ if { $func1_sec == $func2_sec } {
 # Now start a clean GDB, for add-symbol-file tests.
 clean_restart
 
-gdb_test "add-symbol-file ${binfile} 0 -s ${func1_sec} 0x10000 -s ${func2_sec} 0x20000" \
+set host_binfile [gdb_remote_download host $binfile]
+
+gdb_test "add-symbol-file $host_binfile 0 -s ${func1_sec} 0x10000 -s ${func2_sec} 0x20000" \
 	"Reading symbols from .*${testfile}\\.o\\.\\.\\.(|\r\nUsing host libthread_db library .*libthread_db.so.*\\.)" \
 	"add-symbol-file ${testfile}.o" \
 	"add symbol table from file \".*${testfile}\\.o\" at.*\\(y or n\\) " \
-- 
2.35.3


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host
  2023-03-22  8:38 ` [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host Tom de Vries
@ 2023-03-22 15:05   ` Simon Marchi
  2023-03-23  0:18     ` Tom de Vries
  0 siblings, 1 reply; 7+ messages in thread
From: Simon Marchi @ 2023-03-22 15:05 UTC (permalink / raw)
  To: Tom de Vries, gdb-patches



On 3/22/23 04:38, Tom de Vries via Gdb-patches wrote:
> Fix a few test-cases in gdb.cp/*.exp for remote host using new proc
> include_file.

Starting with this patch, I see:

    info variable everywhere^M
    All variables matching regular expression "everywhere":^M
    ^M
    File /home/simark/src/binutils-gdb/gdb/testsuite/gdb.cp/m-static.h:^M
    8:      const int gnu_obj_4::everywhere;^M
    (gdb) FAIL: gdb.cp/m-static.exp: info variable everywhere

Simon

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host
  2023-03-22 15:05   ` Simon Marchi
@ 2023-03-23  0:18     ` Tom de Vries
  2023-03-23 13:49       ` Tom de Vries
  0 siblings, 1 reply; 7+ messages in thread
From: Tom de Vries @ 2023-03-23  0:18 UTC (permalink / raw)
  To: Simon Marchi, gdb-patches

On 3/22/23 16:05, Simon Marchi wrote:
> 
> 
> On 3/22/23 04:38, Tom de Vries via Gdb-patches wrote:
>> Fix a few test-cases in gdb.cp/*.exp for remote host using new proc
>> include_file.
> 
> Starting with this patch, I see:
> 
>      info variable everywhere^M
>      All variables matching regular expression "everywhere":^M
>      ^M
>      File /home/simark/src/binutils-gdb/gdb/testsuite/gdb.cp/m-static.h:^M
>      8:      const int gnu_obj_4::everywhere;^M
>      (gdb) FAIL: gdb.cp/m-static.exp: info variable everywhere
> 

Hi Simon,

thanks for reporting this.

I don't see this on openSUSE Leap 15.4 and Tumbleweed, but I managed to 
reproduce on Ubuntu 20.04 (though with the readnow board, it disappears 
again).

Anyway, the problem is that I changed the matching to be more strict, 
and that's what we're seeing the effect of.  I'll propose a patch that 
reverts things to the way things were before, with less strict matching.

As for what I'm seeing:
...
(gdb) info variable everywhere^M
All variables matching regular expression "everywhere":^M
^M
File /data/vries/gdb/src/gdb/testsuite/gdb.cp/m-static.h:^M
8:      const int gnu_obj_4::everywhere;^M
^M
File /data/vries/gdb/src/gdb/testsuite/gdb.cp/m-static1.cc:^M
8:      const int gnu_obj_4::everywhere;^M
(gdb) PASS: gdb.cp/m-static.exp: info variable everywhere
...
the combination of line 8 with m-static1.cc doesn't seem to make sense.

I don't know this as a known issue, so I'll file a PR about it.

Thanks,
- Tom

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host
  2023-03-23  0:18     ` Tom de Vries
@ 2023-03-23 13:49       ` Tom de Vries
  0 siblings, 0 replies; 7+ messages in thread
From: Tom de Vries @ 2023-03-23 13:49 UTC (permalink / raw)
  To: Simon Marchi, gdb-patches

On 3/23/23 01:18, Tom de Vries wrote:
> On 3/22/23 16:05, Simon Marchi wrote:
>>
>>
>> On 3/22/23 04:38, Tom de Vries via Gdb-patches wrote:
>>> Fix a few test-cases in gdb.cp/*.exp for remote host using new proc
>>> include_file.
>>
>> Starting with this patch, I see:
>>
>>      info variable everywhere^M
>>      All variables matching regular expression "everywhere":^M
>>      ^M
>>      File 
>> /home/simark/src/binutils-gdb/gdb/testsuite/gdb.cp/m-static.h:^M
>>      8:      const int gnu_obj_4::everywhere;^M
>>      (gdb) FAIL: gdb.cp/m-static.exp: info variable everywhere
>>
> 
> Hi Simon,
> 
> thanks for reporting this.
> 
> I don't see this on openSUSE Leap 15.4 and Tumbleweed, but I managed to 
> reproduce on Ubuntu 20.04 (though with the readnow board, it disappears 
> again).
> 
> Anyway, the problem is that I changed the matching to be more strict, 
> and that's what we're seeing the effect of.  I'll propose a patch that 
> reverts things to the way things were before, with less strict matching.
> 

Proposed patch is here ( 
https://sourceware.org/pipermail/gdb-patches/2023-March/198240.html ).

> As for what I'm seeing:
> ...
> (gdb) info variable everywhere^M
> All variables matching regular expression "everywhere":^M
> ^M
> File /data/vries/gdb/src/gdb/testsuite/gdb.cp/m-static.h:^M
> 8:      const int gnu_obj_4::everywhere;^M
> ^M
> File /data/vries/gdb/src/gdb/testsuite/gdb.cp/m-static1.cc:^M
> 8:      const int gnu_obj_4::everywhere;^M
> (gdb) PASS: gdb.cp/m-static.exp: info variable everywhere
> ...
> the combination of line 8 with m-static1.cc doesn't seem to make sense.
> 
> I don't know this as a known issue, so I'll file a PR about it.
> 

Filed this PR ( https://sourceware.org/bugzilla/show_bug.cgi?id=30265 ).

Thanks,
- Tom




^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2023-03-23 13:49 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-22  8:38 [pushed 0/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host/target Tom de Vries
2023-03-22  8:38 ` [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host Tom de Vries
2023-03-22 15:05   ` Simon Marchi
2023-03-23  0:18     ` Tom de Vries
2023-03-23 13:49       ` Tom de Vries
2023-03-22  8:38 ` [pushed 2/3] [gdb/testsuite] Fix gdb.cp/annota{2,3}.exp for native-extended-gdbserver Tom de Vries
2023-03-22  8:38 ` [pushed 3/3] [gdb/testsuite] Fix gdb.cp/cp-relocate.exp for remote host 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).