This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Cleanup mi-async.exp


Pedro has noticed that mi-async.exp test is checking for 'async' target,
which was eliminated quite some time ago. It appears that further, that
tests always runs natively, no matter what target the rest of GDB is
using. So, I've checked in the following patch to clean up things.

- Volodya

Index: ChangeLog                                                            
===================================================================         
RCS file: /cvs/src/src/gdb/testsuite/ChangeLog,v                            
retrieving revision 1.2095                                                  
diff -u -p -r1.2095 ChangeLog                                               
--- ChangeLog   19 Jan 2010 20:31:37 -0000      1.2095                      
+++ ChangeLog   20 Jan 2010 14:53:50 -0000                                  
@@ -1,3 +1,8 @@
+2010-01-20  Vladimir Prus  <vladimir@codesourcery.com>
+
+       * gdb.mi/mi-async.exp: Remove check for 'async' target, because
+       that target was removed, and the test always runs native anyway.
+
 2010-01-19  Jan Kratochvil  <jan.kratochvil@redhat.com>

        * gdb.base/watchpoint-cond-gone.exp, gdb.base/watchpoint-cond-gone.c,
Index: gdb.mi/mi-async.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.mi/mi-async.exp,v
retrieving revision 1.10
diff -u -p -r1.10 mi-async.exp
--- gdb.mi/mi-async.exp 1 Jan 2010 07:32:03 -0000       1.10
+++ gdb.mi/mi-async.exp 20 Jan 2010 14:53:50 -0000
@@ -17,10 +17,11 @@
 # Test asynchronous Machine Interface (MI) responses.
 #

-# This currently only works with native linux and the async remote protocol.
-if { !([isnative] && [istarget *-linux*]) \
-     && (![target_info exists gdb_protocol] \
-         || [target_info gdb_protocol] != "async") } then {
+# Because of the way this test is written, using 'start' and not using
+# mi_run_cmd, it ignores whatever target the rest of GDB testsuite is
+# using, and always tries to run natively.  So, don't do anything unless
+# we're actually testing native.
+if { !([isnative] && [istarget *-linux*]) } then {
   return
 }



- Volodya


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]