From: "Maciej W. Rozycki" <macro@ds2.pg.gda.pl>
To: gdb-patches@sources.redhat.com
Subject: [patch] 6.0: Use cconfig.h instead of config.h in sim/common
Date: Mon, 17 Nov 2003 17:02:00 -0000 [thread overview]
Message-ID: <Pine.LNX.4.55.0311171748320.7006@jurand.ds.pg.gda.pl> (raw)
Hello,
Since files from sim/common are built in a target-dependent directory, an
ambiguity exists in which of the common and the target-dependent
configuration headers are included. Both directories has their own
configure scripts each and both scripts create a local configuration. To
disambiguate between them, config.h in common has been renamed to
cconfig.h, but a few files still refer to config.h. I think they all
should include cconfig.h as it records the results of tests specific to
the common directory, i.e. to these very files. The config.h header is
target-specific and need not correctly provide all examined macros. Am I
missing anything here?
Here is a proposed fix. It adds "#ifdef HAVE_CONFIG_H" for consistency
as well, although it's probably and overkill here.
2003-11-17 Maciej W. Rozycki <macro@ds2.pg.gda.pl>
* callback.c: Include cconfig.h instead of config.h.
* run.c: Likewise.
* sim-basics.h: Likewise.
* sim-load.c: Likewise.
* syscall.c: Likewise.
I've successfully tested it with a mipsel-linux target. Please apply.
Maciej
--
+ Maciej W. Rozycki, Technical University of Gdansk, Poland +
+--------------------------------------------------------------+
+ e-mail: macro@ds2.pg.gda.pl, PGP key available +
gdb-6.0-sim-cconfig.patch
diff -up --recursive --new-file gdb-6.0.macro/sim/common/callback.c gdb-6.0/sim/common/callback.c
--- gdb-6.0.macro/sim/common/callback.c 2002-06-09 15:45:45.000000000 +0000
+++ gdb-6.0/sim/common/callback.c 2003-11-09 01:07:33.000000000 +0000
@@ -22,7 +22,7 @@
level. */
#ifdef HAVE_CONFIG_H
-#include "config.h"
+#include "cconfig.h"
#endif
#include "ansidecl.h"
#ifdef ANSI_PROTOTYPES
diff -up --recursive --new-file gdb-6.0.macro/sim/common/run.c gdb-6.0/sim/common/run.c
--- gdb-6.0.macro/sim/common/run.c 2003-06-05 02:17:29.000000000 +0000
+++ gdb-6.0/sim/common/run.c 2003-11-16 19:09:07.000000000 +0000
@@ -18,8 +18,10 @@ with this program; if not, write to the
/* Steve Chamberlain sac@cygnus.com,
and others at Cygnus. */
-#include "config.h"
+#ifdef HAVE_CONFIG_H
+#include "cconfig.h"
#include "tconfig.h"
+#endif
#include <signal.h>
#include <stdio.h>
diff -up --recursive --new-file gdb-6.0.macro/sim/common/sim-basics.h gdb-6.0/sim/common/sim-basics.h
--- gdb-6.0.macro/sim/common/sim-basics.h 2002-11-23 01:12:05.000000000 +0000
+++ gdb-6.0/sim/common/sim-basics.h 2003-11-09 01:06:45.000000000 +0000
@@ -29,7 +29,7 @@
/* Basic configuration */
#ifdef HAVE_CONFIG_H
-#include "config.h"
+#include "cconfig.h"
#endif
/* Basic host dependant mess - hopefully <stdio.h> + <stdarg.h> will
diff -up --recursive --new-file gdb-6.0.macro/sim/common/sim-load.c gdb-6.0/sim/common/sim-load.c
--- gdb-6.0.macro/sim/common/sim-load.c 2002-06-09 15:45:45.000000000 +0000
+++ gdb-6.0/sim/common/sim-load.c 2003-11-09 01:09:52.000000000 +0000
@@ -19,7 +19,9 @@ along with this program; if not, write t
as it is used by simulators that don't use it [though that doesn't mean
to suggest that they shouldn't :-)]. */
-#include "config.h"
+#ifdef HAVE_CONFIG_H
+#include "cconfig.h"
+#endif
#include "ansidecl.h"
#include <stdio.h> /* for NULL */
#ifdef ANSI_PROTOTYPES
diff -up --recursive --new-file gdb-6.0.macro/sim/common/syscall.c gdb-6.0/sim/common/syscall.c
--- gdb-6.0.macro/sim/common/syscall.c 2002-06-09 15:45:45.000000000 +0000
+++ gdb-6.0/sim/common/syscall.c 2003-11-09 01:10:04.000000000 +0000
@@ -25,7 +25,7 @@
supported. */
#ifdef HAVE_CONFIG_H
-#include "config.h"
+#include "cconfig.h"
#endif
#include "ansidecl.h"
#include "libiberty.h"
next reply other threads:[~2003-11-17 17:02 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-11-17 17:02 Maciej W. Rozycki [this message]
2003-12-02 16:10 ` Daniel Jacobowitz
2003-12-02 17:35 ` Frank Ch. Eigler
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=Pine.LNX.4.55.0311171748320.7006@jurand.ds.pg.gda.pl \
--to=macro@ds2.pg.gda.pl \
--cc=gdb-patches@sources.redhat.com \
/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