Message ID | 1481064953-31780-1-git-send-email-lgustavo@codesourcery.com |
---|---|
State | New, archived |
Headers |
Received: (qmail 84778 invoked by alias); 6 Dec 2016 22:56:08 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: <gdb-patches.sourceware.org> List-Unsubscribe: <mailto:gdb-patches-unsubscribe-##L=##H@sourceware.org> List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org> List-Archive: <http://sourceware.org/ml/gdb-patches/> List-Post: <mailto:gdb-patches@sourceware.org> List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs> Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 84764 invoked by uid 89); 6 Dec 2016 22:56:07 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=AWL, BAYES_00, RCVD_IN_DNSWL_NONE, SPF_PASS, URIBL_RED autolearn=ham version=3.3.2 spammy=Hx-languages-length:1599 X-HELO: relay1.mentorg.com Received: from relay1.mentorg.com (HELO relay1.mentorg.com) (192.94.38.131) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 06 Dec 2016 22:55:57 +0000 Received: from svr-orw-fem-06.mgc.mentorg.com ([147.34.97.120]) by relay1.mentorg.com with esmtp id 1cEOeR-00034w-RE from Luis_Gustavo@mentor.com ; Tue, 06 Dec 2016 14:55:55 -0800 Received: from Opsys.world.mentorg.com (147.34.91.1) by SVR-ORW-FEM-06.mgc.mentorg.com (147.34.97.120) with Microsoft SMTP Server id 14.3.224.2; Tue, 6 Dec 2016 14:55:55 -0800 From: Luis Machado <lgustavo@codesourcery.com> To: <gdb-patches@sourceware.org>, <simon.marchi@polymtl.ca> Subject: Re: [PATCH, obvious?] Fix incorrect reference to source files Date: Tue, 6 Dec 2016 16:55:53 -0600 Message-ID: <1481064953-31780-1-git-send-email-lgustavo@codesourcery.com> In-Reply-To: <2c0de800bcae514b2a8c27c6f47271da@polymtl.ca> References: <2c0de800bcae514b2a8c27c6f47271da@polymtl.ca> MIME-Version: 1.0 Content-Type: text/plain X-IsSubscribed: yes |
Commit Message
Luis Machado
Dec. 6, 2016, 10:55 p.m. UTC
How about the following? I also fixed another occurrences of the same mistake in another source file. ---- gdb/gdbserver/ChangeLog: 2016-12-06 Luis Machado <lgustavo@codesourcery.com> * win32-i386-low.c: Fix incorrect reference to a couple source files. * nto-x86-low.c: Likewise. --- gdb/gdbserver/nto-x86-low.c | 2 +- gdb/gdbserver/win32-i386-low.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-)
Comments
On 2016-12-06 17:55, Luis Machado wrote: > How about the following? I also fixed another occurrences of the same > mistake > in another source file. > > ---- > > gdb/gdbserver/ChangeLog: > > 2016-12-06 Luis Machado <lgustavo@codesourcery.com> > > * win32-i386-low.c: Fix incorrect reference to a couple source files. > * nto-x86-low.c: Likewise. > --- > gdb/gdbserver/nto-x86-low.c | 2 +- > gdb/gdbserver/win32-i386-low.c | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/gdb/gdbserver/nto-x86-low.c b/gdb/gdbserver/nto-x86-low.c > index da77c54..c7303ab 100644 > --- a/gdb/gdbserver/nto-x86-low.c > +++ b/gdb/gdbserver/nto-x86-low.c > @@ -25,7 +25,7 @@ > #include <x86/context.h> > > > -/* Definition auto generated from reg-i386.dep. */ > +/* Defined in auto-generated build-time file gdb/gdbserver/i386.c. */ > extern void init_registers_i386 (); > extern struct reg *regs_i386; > extern const struct target_desc *tdesc_i386; > diff --git a/gdb/gdbserver/win32-i386-low.c > b/gdb/gdbserver/win32-i386-low.c > index 5daa863..dc20d68 100644 > --- a/gdb/gdbserver/win32-i386-low.c > +++ b/gdb/gdbserver/win32-i386-low.c > @@ -29,11 +29,11 @@ > #define FLAG_TRACE_BIT 0x100 > > #ifdef __x86_64__ > -/* Defined in auto-generated file reg-amd64.c. */ > +/* Defined in auto-generated build-time file gdb/gdbserver/amd64.c. > */ > void init_registers_amd64 (void); > extern const struct target_desc *tdesc_amd64; > #else > -/* Defined in auto-generated file reg-i386.c. */ > +/* Defined in auto-generated build-time file gdb/gdbserver/i386.c. */ > void init_registers_i386 (void); > extern const struct target_desc *tdesc_i386; > #endif You used the wrong directory :). gdb/gdbserver -> gdb/features/i386
On 12/06/2016 05:02 PM, Simon Marchi wrote: > On 2016-12-06 17:55, Luis Machado wrote: >> How about the following? I also fixed another occurrences of the same >> mistake >> in another source file. >> >> ---- >> >> gdb/gdbserver/ChangeLog: >> >> 2016-12-06 Luis Machado <lgustavo@codesourcery.com> >> >> * win32-i386-low.c: Fix incorrect reference to a couple source files. >> * nto-x86-low.c: Likewise. >> --- >> gdb/gdbserver/nto-x86-low.c | 2 +- >> gdb/gdbserver/win32-i386-low.c | 4 ++-- >> 2 files changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/gdb/gdbserver/nto-x86-low.c b/gdb/gdbserver/nto-x86-low.c >> index da77c54..c7303ab 100644 >> --- a/gdb/gdbserver/nto-x86-low.c >> +++ b/gdb/gdbserver/nto-x86-low.c >> @@ -25,7 +25,7 @@ >> #include <x86/context.h> >> >> >> -/* Definition auto generated from reg-i386.dep. */ >> +/* Defined in auto-generated build-time file gdb/gdbserver/i386.c. */ >> extern void init_registers_i386 (); >> extern struct reg *regs_i386; >> extern const struct target_desc *tdesc_i386; >> diff --git a/gdb/gdbserver/win32-i386-low.c >> b/gdb/gdbserver/win32-i386-low.c >> index 5daa863..dc20d68 100644 >> --- a/gdb/gdbserver/win32-i386-low.c >> +++ b/gdb/gdbserver/win32-i386-low.c >> @@ -29,11 +29,11 @@ >> #define FLAG_TRACE_BIT 0x100 >> >> #ifdef __x86_64__ >> -/* Defined in auto-generated file reg-amd64.c. */ >> +/* Defined in auto-generated build-time file gdb/gdbserver/amd64.c. */ >> void init_registers_amd64 (void); >> extern const struct target_desc *tdesc_amd64; >> #else >> -/* Defined in auto-generated file reg-i386.c. */ >> +/* Defined in auto-generated build-time file gdb/gdbserver/i386.c. */ >> void init_registers_i386 (void); >> extern const struct target_desc *tdesc_i386; >> #endif > > You used the wrong directory :). > > gdb/gdbserver -> gdb/features/i386 > The files don't live inside gdb/features/, they are auto-generated based on the .dat files from gdb/regformats/ and put inside <build>/gdb/gdbserver/, no?
On 2016-12-06 18:06, Luis Machado wrote: > The files don't live inside gdb/features/, they are auto-generated > based on the .dat files from gdb/regformats/ and put inside > <build>/gdb/gdbserver/, no? Oh ok I got confused with the other files named the same way, sorry. Then if they're all in the same directory, my initial comment may not apply, it might be obvious enough.
On 12/06/2016 05:14 PM, Simon Marchi wrote: > On 2016-12-06 18:06, Luis Machado wrote: >> The files don't live inside gdb/features/, they are auto-generated >> based on the .dat files from gdb/regformats/ and put inside >> <build>/gdb/gdbserver/, no? > > Oh ok I got confused with the other files named the same way, sorry. > Then if they're all in the same directory, my initial comment may not > apply, it might be obvious enough. I'm still happy with the last patch. I think it clarifies things a bit or at least points the user toward the right direction (build directory). Does anyone have objections?
On 12/07/2016 06:57 PM, Luis Machado wrote: > On 12/06/2016 05:14 PM, Simon Marchi wrote: >> On 2016-12-06 18:06, Luis Machado wrote: >>> The files don't live inside gdb/features/, they are auto-generated >>> based on the .dat files from gdb/regformats/ and put inside >>> <build>/gdb/gdbserver/, no? >> >> Oh ok I got confused with the other files named the same way, sorry. >> Then if they're all in the same directory, my initial comment may not >> apply, it might be obvious enough. > > I'm still happy with the last patch. I think it clarifies things a bit > or at least points the user toward the right direction (build directory). > > Does anyone have objections? Fine with me. Thanks, Pedro Alves
On 12/09/2016 11:02 AM, Pedro Alves wrote: > On 12/07/2016 06:57 PM, Luis Machado wrote: >> On 12/06/2016 05:14 PM, Simon Marchi wrote: >>> On 2016-12-06 18:06, Luis Machado wrote: >>>> The files don't live inside gdb/features/, they are auto-generated >>>> based on the .dat files from gdb/regformats/ and put inside >>>> <build>/gdb/gdbserver/, no? >>> >>> Oh ok I got confused with the other files named the same way, sorry. >>> Then if they're all in the same directory, my initial comment may not >>> apply, it might be obvious enough. >> >> I'm still happy with the last patch. I think it clarifies things a bit >> or at least points the user toward the right direction (build directory). >> >> Does anyone have objections? > > Fine with me. > > Thanks, > Pedro Alves > > Thanks. Pushed now.
diff --git a/gdb/gdbserver/nto-x86-low.c b/gdb/gdbserver/nto-x86-low.c index da77c54..c7303ab 100644 --- a/gdb/gdbserver/nto-x86-low.c +++ b/gdb/gdbserver/nto-x86-low.c @@ -25,7 +25,7 @@ #include <x86/context.h> -/* Definition auto generated from reg-i386.dep. */ +/* Defined in auto-generated build-time file gdb/gdbserver/i386.c. */ extern void init_registers_i386 (); extern struct reg *regs_i386; extern const struct target_desc *tdesc_i386; diff --git a/gdb/gdbserver/win32-i386-low.c b/gdb/gdbserver/win32-i386-low.c index 5daa863..dc20d68 100644 --- a/gdb/gdbserver/win32-i386-low.c +++ b/gdb/gdbserver/win32-i386-low.c @@ -29,11 +29,11 @@ #define FLAG_TRACE_BIT 0x100 #ifdef __x86_64__ -/* Defined in auto-generated file reg-amd64.c. */ +/* Defined in auto-generated build-time file gdb/gdbserver/amd64.c. */ void init_registers_amd64 (void); extern const struct target_desc *tdesc_amd64; #else -/* Defined in auto-generated file reg-i386.c. */ +/* Defined in auto-generated build-time file gdb/gdbserver/i386.c. */ void init_registers_i386 (void); extern const struct target_desc *tdesc_i386; #endif