Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [PATCH] gdb/Makefile.in: fix 'make tags' failure
@ 2017-10-30 22:19 Mike Gulick
  2017-10-30 23:02 ` Sergio Durigan Junior
  0 siblings, 1 reply; 5+ messages in thread
From: Mike Gulick @ 2017-10-30 22:19 UTC (permalink / raw)
  To: gdb-patches

'make tags' has been broken in master for a couple of weeks.  Hopefully this
is small enough to not require copyright attribution.  Otherwise someone can
be "inspired" by this change to fix it themselves.

Thanks,
Mike

---
From c3936e0e1bf4fab486965496d7ad3966b1846eb8 Mon Sep 17 00:00:00 2001
From: Mike Gulick <mgulick@mathworks.com>
Date: Thu, 19 Oct 2017 16:51:33 -0400
Subject: [PATCH] gdb/Makefile.in: fix 'make tags' failure

Remove reference to gdb.h, which was removed in commit
65630365f7d073430e62b4fe65f34dcdc0a4b05e.
---
 gdb/Makefile.in | 1 -
 1 file changed, 1 deletion(-)

diff --git a/gdb/Makefile.in b/gdb/Makefile.in
index 6cb0970..1a68746 100644
--- a/gdb/Makefile.in
+++ b/gdb/Makefile.in
@@ -1346,7 +1346,6 @@ HFILES_NO_SRCDIR = \
   frv-tdep.h \
   ft32-tdep.h \
   gcore.h \
-  gdb.h \
   gdb_bfd.h \
   gdb_curses.h \
   gdb-dlfcn.h \
-- 
2.1.4


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] gdb/Makefile.in: fix 'make tags' failure
  2017-10-30 22:19 [PATCH] gdb/Makefile.in: fix 'make tags' failure Mike Gulick
@ 2017-10-30 23:02 ` Sergio Durigan Junior
  2017-10-31  0:12   ` Mike Gulick
  0 siblings, 1 reply; 5+ messages in thread
From: Sergio Durigan Junior @ 2017-10-30 23:02 UTC (permalink / raw)
  To: Mike Gulick; +Cc: gdb-patches

On Monday, October 30 2017, Mike Gulick wrote:

> 'make tags' has been broken in master for a couple of weeks.  Hopefully this
> is small enough to not require copyright attribution.  Otherwise someone can
> be "inspired" by this change to fix it themselves.

Thanks for the patch, Mike.  Could you please write a proper ChangeLog
entry and a more descriptive commit message?  When you do that I can
push this in for you under the obvious rule.

Thanks,

> ---
> From c3936e0e1bf4fab486965496d7ad3966b1846eb8 Mon Sep 17 00:00:00 2001
> From: Mike Gulick <mgulick@mathworks.com>
> Date: Thu, 19 Oct 2017 16:51:33 -0400
> Subject: [PATCH] gdb/Makefile.in: fix 'make tags' failure
>
> Remove reference to gdb.h, which was removed in commit
> 65630365f7d073430e62b4fe65f34dcdc0a4b05e.
> ---
>  gdb/Makefile.in | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/gdb/Makefile.in b/gdb/Makefile.in
> index 6cb0970..1a68746 100644
> --- a/gdb/Makefile.in
> +++ b/gdb/Makefile.in
> @@ -1346,7 +1346,6 @@ HFILES_NO_SRCDIR = \
>    frv-tdep.h \
>    ft32-tdep.h \
>    gcore.h \
> -  gdb.h \
>    gdb_bfd.h \
>    gdb_curses.h \
>    gdb-dlfcn.h \
> -- 
> 2.1.4

-- 
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF  31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] gdb/Makefile.in: fix 'make tags' failure
  2017-10-30 23:02 ` Sergio Durigan Junior
@ 2017-10-31  0:12   ` Mike Gulick
  2017-10-31  1:33     ` Sergio Durigan Junior
  0 siblings, 1 reply; 5+ messages in thread
From: Mike Gulick @ 2017-10-31  0:12 UTC (permalink / raw)
  To: Sergio Durigan Junior; +Cc: gdb-patches

On 10/30/2017 07:02 PM, Sergio Durigan Junior wrote:
> On Monday, October 30 2017, Mike Gulick wrote:
> 
>> 'make tags' has been broken in master for a couple of weeks.  Hopefully this
>> is small enough to not require copyright attribution.  Otherwise someone can
>> be "inspired" by this change to fix it themselves.
> 
> Thanks for the patch, Mike.  Could you please write a proper ChangeLog
> entry and a more descriptive commit message?  When you do that I can
> push this in for you under the obvious rule.

Sure, how about this:

From e330a7a6763a4d520e2c15792154db1aaa30c86a Mon Sep 17 00:00:00 2001
From: Mike Gulick <mgulick@mathworks.com>
Date: Thu, 19 Oct 2017 16:51:33 -0400
Subject: [PATCH] gdb/Makefile.in: fix 'make tags' failure

'make tags' fails with the following error:

  make[2]: Entering directory '/local-ssd/mgulick/gdb/git/binutils-gdb/gdb'
  make[2]: *** No rule to make target 'gdb.h', needed by 'TAGS'.  Stop.
  make[2]: Leaving directory '/local-ssd/mgulick/gdb/git/binutils-gdb/gdb'

The file gdb/gdb.h was removed in commit
65630365f7d073430e62b4fe65f34dcdc0a4b05e.

gdb/ChangeLog:

* Makefile.in (HFILES_NO_SRCDIR): Remove reference to gdb.h.
---
 gdb/ChangeLog   | 4 ++++
 gdb/Makefile.in | 1 -
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 5a680ed..a72cc74 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2017-10-30  Mike Gulick  <mgulick@mathworks.com>
+
+	* Makefile.in (HFILES_NO_SRCDIR): Remove reference to gdb.h.
+
 2017-10-30  Simon Marchi  <simon.marchi@ericsson.com>
 
 	* common/common-utils.h (in_inclusive_range): New function.
diff --git a/gdb/Makefile.in b/gdb/Makefile.in
index 6cb0970..1a68746 100644
--- a/gdb/Makefile.in
+++ b/gdb/Makefile.in
@@ -1346,7 +1346,6 @@ HFILES_NO_SRCDIR = \
 	frv-tdep.h \
 	ft32-tdep.h \
 	gcore.h \
-	gdb.h \
 	gdb_bfd.h \
 	gdb_curses.h \
 	gdb-dlfcn.h \
-- 
2.1.4




^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] gdb/Makefile.in: fix 'make tags' failure
  2017-10-31  0:12   ` Mike Gulick
@ 2017-10-31  1:33     ` Sergio Durigan Junior
  2017-10-31  1:41       ` Mike Gulick
  0 siblings, 1 reply; 5+ messages in thread
From: Sergio Durigan Junior @ 2017-10-31  1:33 UTC (permalink / raw)
  To: Mike Gulick; +Cc: gdb-patches

On Monday, October 30 2017, Mike Gulick wrote:

> On 10/30/2017 07:02 PM, Sergio Durigan Junior wrote:
>> On Monday, October 30 2017, Mike Gulick wrote:
>> 
>>> 'make tags' has been broken in master for a couple of weeks.  Hopefully this
>>> is small enough to not require copyright attribution.  Otherwise someone can
>>> be "inspired" by this change to fix it themselves.
>> 
>> Thanks for the patch, Mike.  Could you please write a proper ChangeLog
>> entry and a more descriptive commit message?  When you do that I can
>> push this in for you under the obvious rule.
>
> Sure, how about this:
>
> From e330a7a6763a4d520e2c15792154db1aaa30c86a Mon Sep 17 00:00:00 2001
> From: Mike Gulick <mgulick@mathworks.com>
> Date: Thu, 19 Oct 2017 16:51:33 -0400
> Subject: [PATCH] gdb/Makefile.in: fix 'make tags' failure
>
> 'make tags' fails with the following error:
>
>   make[2]: Entering directory '/local-ssd/mgulick/gdb/git/binutils-gdb/gdb'
>   make[2]: *** No rule to make target 'gdb.h', needed by 'TAGS'.  Stop.
>   make[2]: Leaving directory '/local-ssd/mgulick/gdb/git/binutils-gdb/gdb'
>
> The file gdb/gdb.h was removed in commit
> 65630365f7d073430e62b4fe65f34dcdc0a4b05e.
>
> gdb/ChangeLog:
>
> * Makefile.in (HFILES_NO_SRCDIR): Remove reference to gdb.h.

This should have been indented with a TAB; I fixed it in the commit log.
Otherwise, looks good.  I went ahead and pushed as obvious:

f871c4853a9725682011c2314ea4308f3c34bd8d

Thanks!

> ---
>  gdb/ChangeLog   | 4 ++++
>  gdb/Makefile.in | 1 -
>  2 files changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/gdb/ChangeLog b/gdb/ChangeLog
> index 5a680ed..a72cc74 100644
> --- a/gdb/ChangeLog
> +++ b/gdb/ChangeLog
> @@ -1,3 +1,7 @@
> +2017-10-30  Mike Gulick  <mgulick@mathworks.com>
> +
> +	* Makefile.in (HFILES_NO_SRCDIR): Remove reference to gdb.h.
> +
>  2017-10-30  Simon Marchi  <simon.marchi@ericsson.com>
>  
>  	* common/common-utils.h (in_inclusive_range): New function.
> diff --git a/gdb/Makefile.in b/gdb/Makefile.in
> index 6cb0970..1a68746 100644
> --- a/gdb/Makefile.in
> +++ b/gdb/Makefile.in
> @@ -1346,7 +1346,6 @@ HFILES_NO_SRCDIR = \
>  	frv-tdep.h \
>  	ft32-tdep.h \
>  	gcore.h \
> -	gdb.h \
>  	gdb_bfd.h \
>  	gdb_curses.h \
>  	gdb-dlfcn.h \
> -- 
> 2.1.4

-- 
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF  31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] gdb/Makefile.in: fix 'make tags' failure
  2017-10-31  1:33     ` Sergio Durigan Junior
@ 2017-10-31  1:41       ` Mike Gulick
  0 siblings, 0 replies; 5+ messages in thread
From: Mike Gulick @ 2017-10-31  1:41 UTC (permalink / raw)
  To: Sergio Durigan Junior; +Cc: gdb-patches



On 10/30/2017 09:33 PM, Sergio Durigan Junior wrote:
> On Monday, October 30 2017, Mike Gulick wrote:
> 
>> ...
>> gdb/ChangeLog:
>>
>> * Makefile.in (HFILES_NO_SRCDIR): Remove reference to gdb.h.
> 
> This should have been indented with a TAB; I fixed it in the commit log.
> Otherwise, looks good.  I went ahead and pushed as obvious:
> 
> f871c4853a9725682011c2314ea4308f3c34bd8d
> 
> Thanks!
> 

Thank you!


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2017-10-31  1:41 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-30 22:19 [PATCH] gdb/Makefile.in: fix 'make tags' failure Mike Gulick
2017-10-30 23:02 ` Sergio Durigan Junior
2017-10-31  0:12   ` Mike Gulick
2017-10-31  1:33     ` Sergio Durigan Junior
2017-10-31  1:41       ` Mike Gulick

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox