[binutils-gdb] Pass internal gdb flags to --configuration invocations

Tom Tromey tromey@sourceware.org
Sat Jan 14 19:49:05 GMT 2023


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

commit 856cd0786ccea2f0b39d36024187c415e0a27d76
Author: Tom Tromey <tom@tromey.com>
Date:   Sat Jan 14 12:42:30 2023 -0700

    Pass internal gdb flags to --configuration invocations
    
    The test suite uses the --configuration flag to feature-test gdb.
    However, when I added this, I neglected to pass the internal gdbflags
    to this, causing an error, which then caused failures in the test
    suite (which would not be seen if you'd ever run "make install").
    
    This patch fixes the bug.  Tested by removing my install tree first,
    to verify that I could reproduce the failure.

Diff:
---
 gdb/testsuite/lib/gdb-guile.exp | 2 +-
 gdb/testsuite/lib/gdb.exp       | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/gdb/testsuite/lib/gdb-guile.exp b/gdb/testsuite/lib/gdb-guile.exp
index f2307372b8b..50189c8ed90 100644
--- a/gdb/testsuite/lib/gdb-guile.exp
+++ b/gdb/testsuite/lib/gdb-guile.exp
@@ -21,7 +21,7 @@ set ghex {[0-9a-f]+}
 # Return a 1 for configurations that support Guile scripting.
 
 gdb_caching_proc allow_guile_tests {
-    set output [exec $::GDB --configuration]
+    set output [remote_exec host $::GDB "$::INTERNAL_GDBFLAGS --configuration"]
     return [expr {[string first "--with-guile" $output] != -1}]
 }
 
diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp
index 5359315667e..68337bd235c 100644
--- a/gdb/testsuite/lib/gdb.exp
+++ b/gdb/testsuite/lib/gdb.exp
@@ -2450,7 +2450,7 @@ proc allow_rust_tests {} {
 # Return a 1 for configurations that support Python scripting.
 
 gdb_caching_proc allow_python_tests {
-    set output [remote_exec host $::GDB --configuration]
+    set output [remote_exec host $::GDB "$::INTERNAL_GDBFLAGS --configuration"]
     return [expr {[string first "--with-python" $output] != -1}]
 }
 
@@ -2575,7 +2575,7 @@ gdb_caching_proc allow_dlmopen_tests {
 # Return 1 if we should allow TUI-related tests.
 
 gdb_caching_proc allow_tui_tests {
-    set output [remote_exec host $::GDB --configuration]
+    set output [remote_exec host $::GDB "$::INTERNAL_GDBFLAGS --configuration"]
     return [expr {[string first "--enable-tui" $output] != -1}]
 }


More information about the Gdb-cvs mailing list