From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 64186 invoked by alias); 21 Dec 2015 10:20:13 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 64078 invoked by uid 89); 21 Dec 2015 10:20:12 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=1.7 required=5.0 tests=AWL,BAYES_40,KAM_LAZY_DOMAIN_SECURITY,RP_MATCHES_RCVD autolearn=no version=3.3.2 spammy=sk:owner@s, ownersourcewareorg, sk:owners, owner@sourceware.org X-HELO: foss.arm.com Received: from foss.arm.com (HELO foss.arm.com) (217.140.101.70) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 21 Dec 2015 10:20:07 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 5C2D23A8; Mon, 21 Dec 2015 02:19:39 -0800 (PST) Received: from SHAWIN202 (unknown [10.164.6.62]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 1AB4D3F24D; Mon, 21 Dec 2015 02:20:04 -0800 (PST) From: "Thomas Preud'homme" To: "'Joel Brobecker'" Cc: References: <000101d13979$d1e0c610$75a25230$@foss.arm.com> <20151218182252.GF29928@adacore.com> In-Reply-To: <20151218182252.GF29928@adacore.com> Subject: RE: [PATCH, ping1] Add missing expect for running a program with CLI jump Date: Mon, 21 Dec 2015 10:20:00 -0000 Message-ID: <002a01d13bd9$29442990$7bcc7cb0$@foss.arm.com> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="----=_NextPart_000_002B_01D13C1C.376F0AB0" X-SW-Source: 2015-12/txt/msg00420.txt.bz2 This is a multipart message in MIME format. ------=_NextPart_000_002B_01D13C1C.376F0AB0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Content-length: 2250 Hi Joel, I hope this time it's fine. I'm also adding it in attachment just in case. Best regards, Thomas diff --git a/gdb/testsuite/lib/mi-support.exp b/gdb/testsuite/lib/mi-suppor= t.exp index 0d17ecb..cc055f6 100644 --- a/gdb/testsuite/lib/mi-support.exp +++ b/gdb/testsuite/lib/mi-support.exp @@ -886,6 +886,9 @@ proc mi_run_cmd_full {use_mi_command args} { # to better handle RUN. send_gdb "jump *$start\n" warning "Using CLI jump command, expect run-to-main FAIL" + gdb_expect { + -re "${run_match}&\"jump \\*${start}\\n\"\[\r\n\]+~\"Continuing at 0x= \[0-9A-Fa-f\]+\\n.\"\[\r\n\]+\^running\[\r\n\]+\\*running,thread-id=3D\"\[^= \"\]+\"\r\n${mi_gdb_prompt}" {} + } return 0 } > -----Original Message----- > From: gdb-patches-owner@sourceware.org [mailto:gdb-patches- > owner@sourceware.org] On Behalf Of Joel Brobecker > Sent: Saturday, December 19, 2015 2:23 AM > To: Thomas Preud'homme > Cc: gdb-patches@sourceware.org > Subject: Re: [PATCH, ping1] Add missing expect for running a program > with CLI jump >=20 > > > 2015-10-15 Thomas Preud'homme > > > > > > > * lib/mi-support.exp (mi_run_cmd_full): Add an expect for the > CLI > > > jump > > > case. >=20 > I think the patch is good, but your mailer did some line folding. > Can you resend? >=20 > Thanks! >=20 > > > > > > > > > diff --git a/gdb/testsuite/lib/mi-support.exp b/gdb/testsuite/lib/mi- > > > support.exp > > > index 0d17ecb..cc055f6 100644 > > > --- a/gdb/testsuite/lib/mi-support.exp > > > +++ b/gdb/testsuite/lib/mi-support.exp > > > @@ -886,6 +886,9 @@ proc mi_run_cmd_full {use_mi_command > args} { > > > # to better handle RUN. > > > send_gdb "jump *$start\n" > > > warning "Using CLI jump command, expect run-to-main FAIL" > > > + gdb_expect { > > > + -re "${run_match}&\"jump > > > \\*${start}\\n\"\[\r\n\]+~\"Continuing at 0x\[0-9A-Fa- > > > f\]+\\n.\"\[\r\n\]+\^running\[\r\n\]+\\*running,thread- > > > id=3D\"\[^\"\]+\"\r\n${mi_gdb_prompt}" {} > > > + } > > > return 0 > > > } > > > > > > > > > With this patch applied, testsuite results become much more stable. > > > > > > Is this ok for the master branch? > > > > > > Best regards, > > > > > > Thomas > > > > > >=20 > -- > Joel ------=_NextPart_000_002B_01D13C1C.376F0AB0 Content-Type: application/octet-stream; name="gdb-runto-expect.patch" Content-Transfer-Encoding: quoted-printable Content-Disposition: attachment; filename="gdb-runto-expect.patch" Content-length: 654 diff --git a/gdb/testsuite/lib/mi-support.exp b/gdb/testsuite/lib/mi-suppor= t.exp=0A= index 0d17ecb..cc055f6 100644=0A= --- a/gdb/testsuite/lib/mi-support.exp=0A= +++ b/gdb/testsuite/lib/mi-support.exp=0A= @@ -886,6 +886,9 @@ proc mi_run_cmd_full {use_mi_command args} {=0A= # to better handle RUN.=0A= send_gdb "jump *$start\n"=0A= warning "Using CLI jump command, expect run-to-main FAIL"=0A= + gdb_expect {=0A= + -re "${run_match}&\"jump \\*${start}\\n\"\[\r\n\]+~\"Continuing at 0x= \[0-9A-Fa-f\]+\\n.\"\[\r\n\]+\^running\[\r\n\]+\\*running,thread-id=3D\"\[^= \"\]+\"\r\n${mi_gdb_prompt}" {}=0A= + }=0A= return 0=0A= }=0A= =20=0A= ------=_NextPart_000_002B_01D13C1C.376F0AB0--