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]

Re: [PATCH 1/3] gdb/testsuite: Fix broken regexp in gdbstub case


On 04/09/2018 04:15 PM, Andrew Burgess wrote:
> 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).

That seems to have been added by:
 https://sourceware.org/ml/gdb-patches/2015-12/msg00357.html

curious how it seems to have worked back then.  I wonder whether
it was GDB's CLI-in-MI output that changed meanwhile?

I'd have appreciated a bit more detail on what is actually broken in
the pattern.  The git log above gives no clue, and the diff is not
exactly easy to read.  I trust that it's now correct though.  Thus, OK.

Thanks,
Pedro Alves


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