diff mbox

make "set debug target" take effect immediately

Message ID 8738dkarhc.fsf@fleche.redhat.com
State Committed
Headers

Commit Message

Tom Tromey July 29, 2014, 7:34 p.m. UTC 
  >>>>> "Yao" == Yao Qi <yao@codesourcery.com> writes:

Yao> Beside this change, we still need to update the pattern to match the
Yao> output of "set debug target 1".  We match "target_resume " nowadays,
Yao> but it doesn't exist in the output at all, because of the recent target
Yao> delegation changes.

Thanks for noticing this.

What do you think of the appended?

Tom

2014-07-29  Tom Tromey  <tromey@redhat.com>

	* gdb.base/sss-bp-on-user-bp-2.exp: Match "to_resume", not
	"target_resume".

Comments

Yao Qi July 30, 2014, 1:16 a.m. UTC | #1 
On 07/30/2014 03:34 AM, Tom Tromey wrote:
> What do you think of the appended?

We need to replace "target_resume" in the comments several lines above
too,

# detect that by looking for 'target_resume (..., step)' in "debug
                              ^^^^^^^^^^^^^
# target" output.

Otherwise, the patch looks good to me.
Tom Tromey July 30, 2014, 1:35 p.m. UTC | #2 
Tom> What do you think of the appended?

Yao> We need to replace "target_resume" in the comments several lines above
Yao> too,

Thanks.  I've fixed it locally.

Tom
diff mbox

Patch

diff --git a/gdb/testsuite/gdb.base/sss-bp-on-user-bp-2.exp b/gdb/testsuite/gdb.base/sss-bp-on-user-bp-2.exp
index a196f68..1a48116 100644
--- a/gdb/testsuite/gdb.base/sss-bp-on-user-bp-2.exp
+++ b/gdb/testsuite/gdb.base/sss-bp-on-user-bp-2.exp
@@ -62,7 +62,7 @@  gdb_test_no_output "set debug target 1"
 set hardware_step 0
 set test "probe target hardware step"
 gdb_test_multiple "si" $test {
-    -re "target_resume \\(\[^\r\n\]+, step, .*$gdb_prompt $" {
+    -re "to_resume \\(\[^\r\n\]+, step, .*$gdb_prompt $" {
 	set hardware_step 1
 	pass $test
     }