This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: Ping: [PATCH] testsuite: Add (extensive) hardware breakpoint testing
- From: Pedro Alves <palves at redhat dot com>
- To: "Maciej W. Rozycki" <macro at codesourcery dot com>
- Cc: gdb-patches at sourceware dot org, Doug Evans <dje at google dot com>, Joel Brobecker <brobecker at adacore dot com>
- Date: Thu, 01 Mar 2012 11:41:45 +0000
- Subject: Re: Ping: [PATCH] testsuite: Add (extensive) hardware breakpoint testing
- References: <alpine.DEB.1.10.1111101340460.4191@tp.orcam.me.uk> <CADPb22Qe6G76gDRkVtXhzOhR0w+3VsC9=-eHt-aM=1w8+V7qPg@mail.gmail.com> <alpine.DEB.1.10.1111111907260.4191@tp.orcam.me.uk> <alpine.DEB.1.10.1112081529330.5354@tp.orcam.me.uk> <alpine.DEB.1.10.1202272056010.4811@tp.orcam.me.uk> <4F4D0B18.5040402@redhat.com> <alpine.DEB.1.10.1202281934340.4811@tp.orcam.me.uk>
On 02/28/2012 07:42 PM, Maciej W. Rozycki wrote:
> On Tue, 28 Feb 2012, Pedro Alves wrote:
>>> +if { [prepare_for_testing hbreak2.exp "break" {break.c break1.c} {debug nowarnings}] } {
>>
>> s/"break"/"hbreak2", so that running this test doesn't overwrite break.exp's binary.
>> More instances of this.
>
> Good point, apparently there is just one other place requiring this
> adjustment.
I've added this to the testcase cookbook at <http://sourceware.org/gdb/wiki/GDBTestcaseCookbook>.
>
> Thanks for your review, here's the resulting update. Any further
> thoughts, anyone?
Not from me. Thanks for doing this.
--
Pedro Alves