Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Michael Snyder <msnyder@vmware.com>
To: "gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
Subject: [RFA] Makefile.in, linux.mh: Move Process Record to NATDEPFILES
Date: Fri, 23 Oct 2009 15:51:00 -0000	[thread overview]
Message-ID: <4AE1CF46.7030106@vmware.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 405 bytes --]

Hey folks, we ran into a bunch of build problems because record.c
was being compiled in a lot of builds where it wasn't needed (or
tested).

This change will make record.c be like gcore.c, in that it is only
built if the target config files explicitly call for it.

For this patch, I've only included record.c for i386-linux.
We can add amd64-linux in a separate patch if we decide it is
ready.

Michael


[-- Attachment #2: natdep.txt --]
[-- Type: text/plain, Size: 1716 bytes --]

2009-10-23  Michael Snyder  <msnyder@vmware.com>

	* Makefile.in (COMMON_OBS): Remove record.o.
	(ALL_TARGET_OBS): Remove linux-record.o.
	config/i386/linux.h (NATDEPFILES): Add record.o, linux-record.o.

Index: Makefile.in
===================================================================
RCS file: /cvs/src/src/gdb/Makefile.in,v
retrieving revision 1.1103
diff -u -p -w -b -r1.1103 Makefile.in
--- Makefile.in	21 Oct 2009 14:14:56 -0000	1.1103
+++ Makefile.in	23 Oct 2009 15:45:31 -0000
@@ -531,8 +531,7 @@ ALL_TARGET_OBS = \
 	xcoffread.o \
 	symfile-mem.o \
 	corelow.o \
-	windows-tdep.o \
-	linux-record.o
+	windows-tdep.o
 
 # Host-dependent makefile fragment comes in here.
 @host_makefile_frag@
@@ -834,7 +833,7 @@ COMMON_OBS = $(DEPFILES) $(CONFIG_OBS) $
 	solib.o solib-null.o \
 	prologue-value.o memory-map.o xml-support.o xml-syscall.o \
 	target-descriptions.o target-memory.o xml-tdesc.o xml-builtin.o \
-	inferior.o osdata.o gdb_usleep.o record.o \
+	inferior.o osdata.o gdb_usleep.o \
 	jit.o progspace.o
 
 # Definitions for the syscall's XML files and dir
Index: config/i386/linux.mh
===================================================================
RCS file: /cvs/src/src/gdb/config/i386/linux.mh,v
retrieving revision 1.21
diff -u -p -w -b -r1.21 linux.mh
--- config/i386/linux.mh	14 Aug 2007 12:51:59 -0000	1.21
+++ config/i386/linux.mh	23 Oct 2009 15:45:31 -0000
@@ -4,7 +4,7 @@ NAT_FILE= nm-linux.h
 NATDEPFILES= inf-ptrace.o fork-child.o \
 	i386-nat.o i386-linux-nat.o \
 	proc-service.o linux-thread-db.o gcore.o \
-	linux-nat.o linux-fork.o
+	linux-nat.o linux-fork.o linux-record.o record.o
 
 # The dynamically loaded libthread_db needs access to symbols in the
 # gdb executable.

             reply	other threads:[~2009-10-23 15:51 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-10-23 15:51 Michael Snyder [this message]
2009-10-23 16:11 ` Pedro Alves
2009-10-23 23:44   ` Michael Snyder
2009-10-25  1:38   ` Michael Snyder
2009-10-25  8:12     ` Pedro Alves
2009-10-25 18:41       ` Michael Snyder
2009-10-24  3:07 ` Hui Zhu

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=4AE1CF46.7030106@vmware.com \
    --to=msnyder@vmware.com \
    --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