public inbox for gdb-cvs@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] gdb/Makefile: add some missing config.status dependencies
@ 2024-04-08  9:27 Andrew Burgess
  0 siblings, 0 replies; only message in thread
From: Andrew Burgess @ 2024-04-08  9:27 UTC (permalink / raw)
  To: gdb-cvs

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=cb1bef582e37d29281a5a5ebd99605d55afb16a0

commit cb1bef582e37d29281a5a5ebd99605d55afb16a0
Author: Andrew Burgess <aburgess@redhat.com>
Date:   Fri Apr 5 10:52:17 2024 +0100

    gdb/Makefile: add some missing config.status dependencies
    
    I noticed that for the build targets jit-reader.h, gcore, gdb-gdb.py,
    and gdb-gdb.gdb the rules all use the config.status script, but don't
    have a dependency on the config.status target.  This means we might
    fail to regenerate these targets in a case where config.status, or one
    of its dependencies changes.
    
    Two other targets that use config.status do correctly have a
    dependency on config.status.
    
    Fixed in this commit by adding the missing dependencies.
    
    There should be no changes in _what_ is generated after this commit.
    
    Approved-By: Simon Marchi <simon.marchi@efficios.com>

Diff:
---
 gdb/Makefile.in | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/gdb/Makefile.in b/gdb/Makefile.in
index 9340becbdc9..c1b3144b175 100644
--- a/gdb/Makefile.in
+++ b/gdb/Makefile.in
@@ -2313,16 +2313,16 @@ Makefile: Makefile.in config.status
 run: Makefile
 	./gdb$(EXEEXT) --data-directory=`pwd`/data-directory $(GDBFLAGS)
 
-jit-reader.h: $(srcdir)/jit-reader.in
+jit-reader.h: $(srcdir)/jit-reader.in config.status
 	$(SHELL) config.status $@
 
-gcore: $(srcdir)/gcore.in
+gcore: $(srcdir)/gcore.in config.status
 	$(SHELL) config.status $@
 
-gdb-gdb.py: $(srcdir)/gdb-gdb.py.in
+gdb-gdb.py: $(srcdir)/gdb-gdb.py.in config.status
 	$(SHELL) config.status $@
 
-gdb-gdb.gdb: $(srcdir)/gdb-gdb.gdb.in
+gdb-gdb.gdb: $(srcdir)/gdb-gdb.gdb.in config.status
 	$(SHELL) config.status $@
 
 config.h: stamp-h ; @true

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2024-04-08  9:27 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-04-08  9:27 [binutils-gdb] gdb/Makefile: add some missing config.status dependencies Andrew Burgess

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).