gdb/testsuite: Fix broken regexp in gdbstub case
authorAndrew Burgess <andrew.burgess@embecosm.com>
Thu, 15 Mar 2018 19:41:50 +0000 (19:41 +0000)
committerAndrew Burgess <andrew.burgess@embecosm.com>
Fri, 4 May 2018 10:11:45 +0000 (11:11 +0100)
commit089a949083dcd5c679376033e975d54093007408
treefb5517fb4156537136e1f79b43cf223cd0b28064
parent602f16570454a1597c2af28af66852133432d1f2
gdb/testsuite: Fix broken regexp in gdbstub case

When $use_gdb_stub is true then, when we start an MI target there's a
regexp to match GDB's startup pattern.  Unfortunately the pattern is
broken, and we're also missing a timeout case in the match list (which
would have helped point out that the regexp was broken).

The changes to the regexp are:

  1. Remove '${run_match}' prefix, the issued command doesn't include
  '${run_prefix}' so expecting '${run_match}' is wrong.

  2. Replaced '\\n' with '\\\\n' in order to match literal '\n' in
  GDBs output (that is, match a backslash followed by 'n', not a
  newline character).

  3. Replaced a '.' (matching any character) with '\.' to match a '.'
  and moved the '\.' into the correct place in the regexp.

  4. Replaced '\r\n' with '[\r\n]+' to match the end of a line.  This
  change isn't esential, but matches the other end of line patterns
  within this regexp.

Here's an example of the output that the regexp should match taken
from a testfile log, the first line is the command sent to GDB, and
the remaining lines are the response from GDB:

  jump *_start
  &"jump *_start\n"
  ~"Continuing at 0x10074.\n"
  ^running
  *running,thread-id="all"
  (gdb)

gdb/testsuite/ChangeLog:

* lib/mi-support.exp (mi_run_cmd_full): Fix regexp and add a
timeout.
gdb/testsuite/ChangeLog
gdb/testsuite/lib/mi-support.exp