This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[PING][PATCH] GDB/testsuite: Add a way to send multiple init commands
- From: "Maciej W. Rozycki" <macro at codesourcery dot com>
- To: Tom Tromey <tromey at redhat dot com>
- Cc: Keith Seitz <keiths at redhat dot com>, Eli Zaretskii <eliz at gnu dot org>, <gdb-patches at sourceware dot org>
- Date: Fri, 20 Jun 2014 00:39:32 +0100
- Subject: [PING][PATCH] GDB/testsuite: Add a way to send multiple init commands
- Authentication-results: sourceware.org; auth=none
- References: <alpine dot DEB dot 1 dot 10 dot 1406091614210 dot 3047 at tp dot orcam dot me dot uk> <87a99jwj4u dot fsf at fleche dot redhat dot com> <alpine dot DEB dot 1 dot 10 dot 1406111821290 dot 3047 at tp dot orcam dot me dot uk>
On Wed, 11 Jun 2014, Maciej W. Rozycki wrote:
> On Wed, 11 Jun 2014, Tom Tromey wrote:
>
> > Maciej> 2014-06-10 Maciej W. Rozycki <macro@mips.com>
> > Maciej> Maciej W. Rozycki <macro@codesourcery.com>
> >
> > Maciej> * lib/gdb.exp (gdb_run_cmd): Process `gdb_init_commands'.
> > Maciej> * lib/mi-support.exp (mi_run_cmd): Process `gdb_init_commands'.
> >
> > I like Keith's proposed cleanup.
> >
> > Maciej> + set commands [target_info gdb_init_commands];
> >
> > Extraneous ";". There are a few of these.
>
> Sigh, it just shows my lack of experience with TCL back in 2005. I've
> been pondering an update to the change to avoid some code duplication, so
> given I had to touch it anyway I went ahead and did it. Here's the
> result.
>
> It has been lightly tested with gdb.base/advance.exp, with neither
> setting defined, with `gdb_init_command' only, with `gdb_init_commands'
> only having a single element, with `gdb_init_commands' only having two
> elements and with both `gdb_init_command' and `gdb_init_commands' defined,
> the latter having two elements. It has been also smoke-tested with
> gdb.mi/mi-break.exp, with the last arrangement mentioned above only.
>
> > I think this patch should also update testsuite/README to document the
> > new setting.
>
> And presumably the old one as well, right? Cc-ing Eli for this part.
>
> 2014-06-11 Maciej W. Rozycki <macro@mips.com>
> Maciej W. Rozycki <macro@codesourcery.com>
>
> * lib/gdb.exp (gdb_run_cmd): Process `gdb_init_commands'.
> * lib/mi-support.exp (mi_run_cmd): Process `gdb_init_commands'.
> * README (Board Settings): Document `gdb_init_command' and
> `gdb_init_commands'.
Ping.
Maciej