From: Patrick Monnerat <patrick@monnerat.net>
To: gdb-patches@sourceware.org
Cc: Patrick Monnerat <patrick@monnerat.net>
Subject: [PUSHED] gdb: store gdbtk object files into their sources directory
Date: Sat, 11 Apr 2026 16:11:09 +0200 [thread overview]
Message-ID: <20260411141108.78765-2-patrick@monnerat.net> (raw)
Do not list gdbtk required object file names, but generates them from
source file names.
Use gdb object file list rather than static library for linking insight.
---
gdb/Makefile.in | 31 ++++++++++---------------------
1 file changed, 10 insertions(+), 21 deletions(-)
diff --git a/gdb/Makefile.in b/gdb/Makefile.in
index 81a601dc448..b235edf6356 100644
--- a/gdb/Makefile.in
+++ b/gdb/Makefile.in
@@ -559,29 +559,19 @@ GDBTK_LIBRARY = $(datadir)/insight$(GDBTK_VERSION)
# the testsuite won't run properly.
GDBTK_SRC_DIR = @GDBTK_SRC_DIR@
-SUBDIR_GDBTK_OBS = \
- gdbtk.o \
- gdbtk-bp.o \
- gdbtk-cmds.o \
- gdbtk-hooks.o \
- gdbtk-interp.o \
- gdbtk-register.o \
- gdbtk-stack.o \
- gdbtk-varobj.o \
- gdbtk-wrapper.o
-
SUBDIR_GDBTK_SRCS = \
gdbtk/generic/gdbtk.c \
gdbtk/generic/gdbtk-bp.c \
gdbtk/generic/gdbtk-cmds.c \
gdbtk/generic/gdbtk-hooks.c \
gdbtk/generic/gdbtk-interp.c \
- gdbtk/generic/gdbtk-main.c \
gdbtk/generic/gdbtk-register.c \
gdbtk/generic/gdbtk-stack.c \
gdbtk/generic/gdbtk-varobj.c \
gdbtk/generic/gdbtk-wrapper.c
+SUBDIR_GDBTK_OBS = $(patsubst %.c,%.o,$(SUBDIR_GDBTK_SRCS))
+
SUBDIR_GDBTK_DEPS = $(LIBGUI) $(TCL_DEPS) $(TK_DEPS)
SUBDIR_GDBTK_LDFLAGS =
SUBDIR_GDBTK_CFLAGS= -DGDBTK -D@TCL_PLATFORM_DEFINE@ -D@TK_PLATFORM_DEFINE@
@@ -599,7 +589,7 @@ HAVE_NATIVE_GCORE_TARGET = @HAVE_NATIVE_GCORE_TARGET@
HAVE_GSTACK = @HAVE_GSTACK@
CONFIG_SRC_SUBDIR = arch cli dwarf2 mi compile tui unittests guile python \
- target nat
+ target nat gdbtk/generic
CONFIG_DEP_SUBDIR = $(addsuffix /$(DEPDIR),$(CONFIG_SRC_SUBDIR))
# -I. for config files.
@@ -2110,10 +2100,8 @@ $(CONFIG_DEP_SUBDIR):
# Python files need special flags.
python/%.o: INTERNAL_CFLAGS += $(PYTHON_CFLAGS)
-# Rules for compiling .c files in the various source subdirectories.
-%.o: $(srcdir)/gdbtk/generic/%.c
- $(COMPILE) $(all_gdbtk_cflags) $<
- $(POSTCOMPILE)
+# Insight UI needs special flags.
+gdbtk/generic/%.o: INTERNAL_CFLAGS += $(all_gdbtk_cflags)
installcheck:
@@ -2343,6 +2331,7 @@ uninstall: force $(CONFIG_UNINSTALL)
# computing the list of source files from the list of object files.
INIT_FILES_FILTER_OUT = \
+ gdbres.o \
init.o \
version.o \
xml-builtin.o \
@@ -2824,10 +2813,10 @@ clean-gdbtk:
rm -f insight$(EXEEXT)
# Removing the old gdb first works better if it is running, at least on SunOS.
-insight$(EXEEXT): gdbtk-main.o libgdb.a $(CDEPS) $(TDEPLIBS)
- rm -f insight$(EXEEXT)
+insight$(EXEEXT): gdbtk/generic/gdbtk-main.o $(LIBGDB_OBS) $(CDEPS) $(TDEPLIBS)
+ $(SILENCE) rm -f insight$(EXEEXT)
$(ECHO_CXXLD) $(CC_LD) $(INTERNAL_LDFLAGS) $(WIN32LDAPP) \
- -o insight$(EXEEXT) gdbtk-main.o libgdb.a \
+ -o insight$(EXEEXT) gdbtk/generic/gdbtk-main.o $(LIBGDB_OBS) \
$(TDEPLIBS) $(TUI_LIBRARY) $(CLIBS) $(LOADLIBES)
gdbres.o: $(srcdir)/gdbtk/gdb.rc $(srcdir)/gdbtk/gdbtool.ico
@@ -2862,7 +2851,7 @@ endif
# A list of all the objects we might care about in this build, for
# dependency tracking.
-all_object_files = gdb.o $(LIBGDB_OBS) gdbtk-main.o
+all_object_files = gdb.o $(LIBGDB_OBS) gdbtk/generic/gdbtk-main.o
# All the .deps files to include.
all_deps_files = $(foreach dep,$(patsubst %.o,%.Po,$(all_object_files)),\
--
2.53.0
next reply other threads:[~2026-04-11 14:13 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-04-11 14:11 Patrick Monnerat [this message]
2026-06-07 6:19 ` Tom de Vries
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260411141108.78765-2-patrick@monnerat.net \
--to=patrick@monnerat.net \
--cc=gdb-patches@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox