This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [boehm-gc] Fix libjava "PACKAGE_* redefined" errors
- From: Bryce McKinlay <mckinlay at redhat dot com>
- To: Java Patches <java-patches at gcc dot gnu dot org>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Sat, 27 Nov 2004 12:47:12 -0500
- Subject: Re: [boehm-gc] Fix libjava "PACKAGE_* redefined" errors
- References: <41A6BF76.2060705@redhat.com>
Due to a typo, I checked in the wrong gc_config.h.in. Here's the
correct version, which I'm checking in now.
Bryce
2004-11-27 Bryce McKinlay <mckinlay@redhat.com>
* include/gc_config.h.in: Add comment. Define only
THREAD_LOCAL_ALLOC.
Index: include/gc_config.h.in
===================================================================
RCS file: /cvs/gcc/gcc/boehm-gc/include/gc_config.h.in,v
retrieving revision 1.3
diff -u -r1.3 gc_config.h.in
--- include/gc_config.h.in 26 Nov 2004 05:31:48 -0000 1.3
+++ include/gc_config.h.in 27 Nov 2004 17:43:49 -0000
@@ -1,3 +1,7 @@
-#undef ALL_INTERIOR_POINTERS
-#undef GC_GCJ_SUPPORT
+/* gc_config.h. This autoconf header contains macro definitions that
are
+used by GC clients, such as libjava.
+This file is not generated by autoheader. We only want to define what
we
+really need in order to avoid namespace pollution. */
+
+#undef THREAD_LOCAL_ALLOC
On Nov 26, 2004, at 12:30 AM, Bryce McKinlay wrote:
I'm checking in this patch to fix the redefinition errors that occur
in the libjava build.
As far as I can tell libjava only requires one definition from
boehm-gc, so I've moved only that one to gc_config.h and added a
config.h.in to keep autoheader and gc.h happy.
Probably, we should also get rid of libjava's use of
THREAD_LOCAL_ALLOC, and simply assume that its always on?