From: Nathan Sidwell <nathan@codesourcery.com>
To: gdb-patches@sourceware.org
Subject: trace tests & m68k
Date: Mon, 27 Nov 2006 09:43:00 -0000 [thread overview]
Message-ID: <456AB38A.9030504@codesourcery.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 776 bytes --]
All the trace tests in gdb.trace have special case code for [m68k-*-elf], but
AFAICT that's only applicable for some particular m68k implementation. IMHO it
would be far better to have some board specific switch to enable this specific
behaviour, rather than key off the target name. The current test is breaking
for our coldfire testing.
There also appears to be some unterminated string constants in the tests, which
causes problem too.
Now, this patch simply disables the m68k specific code, but I'm open to
suggestions on what board specific variable should be tested -- perhaps
d490_binfile?
nathan
--
Nathan Sidwell :: http://www.codesourcery.com :: CodeSourcery
nathan@codesourcery.com :: http://www.planetfall.pwp.blueyonder.co.uk
[-- Attachment #2: trace.patch --]
[-- Type: text/x-patch, Size: 11246 bytes --]
2006-11-23 Nathan Sidwell <nathan@codesourcery.com>
* gdb.trace/save-trace.exp, gdb.trace/collection.exp,
gdb.trace/while-stepping.exp, gdb.trace/while-dyn.exp,
gdb.trace/actions.exp, gdb.trace/limits.exp,
gdb.trace/passc-dyn.exp, gdb.trace/tfind.exp,
gdb.trace/packetlen.exp, gdb.trace/infotrace.exp,
gdb.trace/report.exp, gdb.trace/passcount.exp,
gdb.trace/backtrace.exp, gdb.trace/circ.exp,
gdb.trace/tracecmd.exp, gdb.trace/deltrace.exp: Disable m68k-elf
specific expectations. Remove unterminated strings used as
comments.
Index: gdb/testsuite/gdb.trace/save-trace.exp
===================================================================
--- gdb/testsuite/gdb.trace/save-trace.exp (revision 155657)
+++ gdb/testsuite/gdb.trace/save-trace.exp (working copy)
@@ -31,7 +31,7 @@ set bug_id 0
gdb_exit
gdb_start
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
set srcfile gdb_c_test.c
set binfile [board_info target d490_binfile];
} else {
Index: gdb/testsuite/gdb.trace/collection.exp
===================================================================
--- gdb/testsuite/gdb.trace/collection.exp (revision 155657)
+++ gdb/testsuite/gdb.trace/collection.exp (working copy)
@@ -17,7 +17,7 @@
# Please email any bugs, comments, and/or additions to this file to:
# bug-gdb@prep.ai.mit.edu
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
pass "Test not supported on this target"
return;
}
Index: gdb/testsuite/gdb.trace/while-stepping.exp
===================================================================
--- gdb/testsuite/gdb.trace/while-stepping.exp (revision 155657)
+++ gdb/testsuite/gdb.trace/while-stepping.exp (working copy)
@@ -31,7 +31,7 @@ set bug_id 0
gdb_exit
gdb_start
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
set srcfile gdb_c_test.c
set binfile [board_info target d490_binfile];
} else {
Index: gdb/testsuite/gdb.trace/actions.exp
===================================================================
--- gdb/testsuite/gdb.trace/actions.exp (revision 155657)
+++ gdb/testsuite/gdb.trace/actions.exp (working copy)
@@ -31,7 +31,7 @@ set bug_id 0
gdb_exit
gdb_start
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
set binfile [board_info target d490_binfile];
set srcfile gdb_c_test.c
} else {
Index: gdb/testsuite/gdb.trace/while-dyn.exp
===================================================================
--- gdb/testsuite/gdb.trace/while-dyn.exp (revision 155657)
+++ gdb/testsuite/gdb.trace/while-dyn.exp (working copy)
@@ -31,7 +31,7 @@ set bug_id 0
gdb_exit
gdb_start
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
load_lib "emc-support.exp"
set testfile "gdb_c_test"
set srcfile $testfile.c
@@ -47,7 +47,6 @@ if [istarget "m68k-*-elf"] then {
-re "MIS-MATCHED.*$gdb_prompt $" {
untested while-dyn.exp
return -1
- all tests in this module will fail.";
}
-re ".*$gdb_prompt $" { }
}
@@ -99,7 +98,7 @@ gdb_trace_setactions "5.12: define while
gdb_test "tstart" "" ""
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
gdb_emclaptop_command "85,1,2,3,4,5,6"
sleep 5
} else {
Index: gdb/testsuite/gdb.trace/passc-dyn.exp
===================================================================
--- gdb/testsuite/gdb.trace/passc-dyn.exp (revision 155657)
+++ gdb/testsuite/gdb.trace/passc-dyn.exp (working copy)
@@ -30,7 +30,7 @@ set bug_id 0
gdb_exit
gdb_start
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
load_lib "emc-support.exp";
set srcfile gdb_c_test.c
set binfile [board_info target d490_binfile];
@@ -45,7 +45,6 @@ if [istarget "m68k-*-elf"] then {
-re "MIS-MATCHED.*$gdb_prompt $" {
untested passc-dyn.exp
return -1
- all tests in this module will fail.";
}
-re ".*$gdb_prompt $" { }
}
@@ -121,7 +120,7 @@ gdb_test "passcount 3 $tdp4" "Setting tr
gdb_test "tstart" "" ""
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
gdb_emclaptop_command "85,1,2,3,4,5,6"
sleep 5
gdb_emclaptop_command "85,7,8,9,A,B,C"
Index: gdb/testsuite/gdb.trace/limits.exp
===================================================================
--- gdb/testsuite/gdb.trace/limits.exp (revision 155657)
+++ gdb/testsuite/gdb.trace/limits.exp (working copy)
@@ -17,7 +17,7 @@
# Please email any bugs, comments, and/or additions to this file to:
# bug-gdb@prep.ai.mit.edu
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
pass "Test not supported on this target"
return;
}
Index: gdb/testsuite/gdb.trace/tfind.exp
===================================================================
--- gdb/testsuite/gdb.trace/tfind.exp (revision 155657)
+++ gdb/testsuite/gdb.trace/tfind.exp (working copy)
@@ -31,7 +31,7 @@ set bug_id 0
gdb_exit
gdb_start
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
load_lib "emc-support.exp";
set srcfile gdb_c_test.c
set binfile [board_info target d490_binfile];
@@ -46,7 +46,6 @@ if [istarget "m68k-*-elf"] then {
-re "MIS-MATCHED.*$gdb_prompt $" {
untested tfind.exp
return -1
- all tests in this module will fail.";
}
-re ".*$gdb_prompt $" { }
}
@@ -138,7 +137,7 @@ gdb_test "tstatus" "\[Tt\]race is runnin
# 6.2 test help tstart
gdb_test "help tstart" "Start trace data collection." "6.2: help tstart"
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
gdb_emclaptop_command "85,1,2,3,4,5,6"
sleep 5
Index: gdb/testsuite/gdb.trace/packetlen.exp
===================================================================
--- gdb/testsuite/gdb.trace/packetlen.exp (revision 155657)
+++ gdb/testsuite/gdb.trace/packetlen.exp (working copy)
@@ -31,7 +31,7 @@ set bug_id 0
gdb_exit
gdb_start
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
load_lib "emc-support.exp"
set srcfile gdb_c_test.c
set binfile [board_info target d490_binfile];
@@ -46,7 +46,6 @@ if [istarget "m68k-*-elf"] then {
-re "MIS-MATCHED.*$gdb_prompt $" {
untested packetlen.exp
return -1
- all tests in this module will fail.";
}
-re ".*$gdb_prompt $" { }
}
@@ -88,7 +87,7 @@ gdb_trace_setactions "setup collect acti
"end" ""
gdb_test "tstart" "" "survive the long packet send"
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
gdb_emclaptop_command "85,1,2,3,4,5,6"
sleep 5
} else {
Index: gdb/testsuite/gdb.trace/infotrace.exp
===================================================================
--- gdb/testsuite/gdb.trace/infotrace.exp (revision 155657)
+++ gdb/testsuite/gdb.trace/infotrace.exp (working copy)
@@ -31,7 +31,7 @@ set bug_id 0
gdb_exit
gdb_start
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
set srcfile gdb_c_test.c
set binfile [board_info target d490_binfile];
} else {
Index: gdb/testsuite/gdb.trace/report.exp
===================================================================
--- gdb/testsuite/gdb.trace/report.exp (revision 155657)
+++ gdb/testsuite/gdb.trace/report.exp (working copy)
@@ -31,7 +31,7 @@ set bug_id 0
gdb_exit
gdb_start
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
load_lib "emc-support.exp";
set srcfile gdb_c_test.c
set binfile [board_info target d490_binfile];
@@ -46,7 +46,6 @@ if [istarget "m68k-*-elf"] then {
-re "MIS-MATCHED.*$gdb_prompt $" {
untested report.exp
return -1
- all tests in this module will fail.";
}
-re ".*$gdb_prompt $" { }
}
@@ -133,13 +132,11 @@ gdb_expect {
if { ($testline1 == 0) || ($testline2 == 0) || ($testline3 == 0) || ($testline4 == 0) || ($testline5 == 0) || ($testline6 == 0) } {
untested report.exp
return -1
-all tests in this module will fail."
}
}
default {
untested report.exp
return -1
-all tests in this module will fail."
}
}
@@ -190,7 +187,7 @@ gdb_trace_setactions "9.x: setup TP to c
gdb_test "tstart" "" ""
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
gdb_emclaptop_command "85,$arg1,$arg2,$arg3,$arg4,$arg5,$arg6"
sleep 5
Index: gdb/testsuite/gdb.trace/passcount.exp
===================================================================
--- gdb/testsuite/gdb.trace/passcount.exp (revision 155657)
+++ gdb/testsuite/gdb.trace/passcount.exp (working copy)
@@ -30,7 +30,7 @@ set bug_id 0
gdb_exit
gdb_start
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
set srcfile gdb_c_test.c
set binfile [board_info target d490_binfile];
} else {
Index: gdb/testsuite/gdb.trace/backtrace.exp
===================================================================
--- gdb/testsuite/gdb.trace/backtrace.exp (revision 155657)
+++ gdb/testsuite/gdb.trace/backtrace.exp (working copy)
@@ -31,7 +31,7 @@ set bug_id 0
gdb_exit
gdb_start
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
load_lib "emc-support.exp";
set srcfile gdb_c_test.c
set binfile [board_info target d490_binfile];
@@ -46,7 +46,6 @@ if [istarget "m68k-*-elf"] then {
-re "MIS-MATCHED.*$gdb_prompt $" {
untested backtrace.exp
return -1
- all tests in this module will fail.";
}
-re ".*$gdb_prompt $" { }
}
@@ -177,7 +176,7 @@ gdb_trace_setactions "8.6: setup TP to c
gdb_test "tstart" "" ""
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
gdb_emclaptop_command "85,$arg1,$arg2,$arg3,$arg4,$arg5,$arg6"
sleep 5
} else {
Index: gdb/testsuite/gdb.trace/circ.exp
===================================================================
--- gdb/testsuite/gdb.trace/circ.exp (revision 155657)
+++ gdb/testsuite/gdb.trace/circ.exp (working copy)
@@ -17,7 +17,7 @@
# Please email any bugs, comments, and/or additions to this file to:
# bug-gdb@prep.ai.mit.edu
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
pass "Test not supported on this target"
return;
}
Index: gdb/testsuite/gdb.trace/tracecmd.exp
===================================================================
--- gdb/testsuite/gdb.trace/tracecmd.exp (revision 155657)
+++ gdb/testsuite/gdb.trace/tracecmd.exp (working copy)
@@ -30,7 +30,7 @@ set bug_id 0
gdb_exit
gdb_start
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
set srcfile gdb_c_test.c
set binfile [board_info target d490_binfile];
} else {
Index: gdb/testsuite/gdb.trace/deltrace.exp
===================================================================
--- gdb/testsuite/gdb.trace/deltrace.exp (revision 155657)
+++ gdb/testsuite/gdb.trace/deltrace.exp (working copy)
@@ -31,7 +31,7 @@ set bug_id 0
gdb_exit
gdb_start
-if [istarget "m68k-*-elf"] then {
+if { 0 && [istarget "m68k-*-elf"] } then {
set srcfile gdb_c_test.c
set binfile [board_info target d490_binfile];
} else {
next reply other threads:[~2006-11-27 9:43 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-11-27 9:43 Nathan Sidwell [this message]
2006-11-27 14:08 ` Daniel Jacobowitz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=456AB38A.9030504@codesourcery.com \
--to=nathan@codesourcery.com \
--cc=gdb-patches@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox