[Gc] bdw-gc.pc: Cflags: -I${includedir}/pc ?

Petter Urkedal urkedal at nbi.dk
Sun Aug 26 14:33:17 PDT 2007


On 2007-08-24, Boehm, Hans wrote:
> I probably made the change that caused gc.h to no longer be installed in
> $(prefix)/include.  But this was not intentional.  I'm also not a
> pkg-config expert or an expert on the conventions here.  Thus I'm very
> open to suggestions/patches.

In my opinion, keep the current target installation directory. To
accommodate package writers and end users who just what the current code
to compile, I think adding <gc.h> and <gc_cpp.h> (which just includes
<gc/gc.h> and <gc/gc_cpp.h>) covers the most common use.  My suggested
patch is below.  If the plan is to get rid of these files in the future,
also put in

#if defined(__GNU_C__)
#  warning This header will be removed in a future version.
#  warning Please use <gc/gc.h>.
#endif


diff --git a/include/extra/gc.h b/include/extra/gc.h
new file mode 100644
index 0000000..55ae4c6
--- /dev/null
+++ b/include/extra/gc.h
@@ -0,0 +1,2 @@
+/* This file is installed for backward compatibility. */
+#include <gc/gc.h>
diff --git a/include/extra/gc_cpp.h b/include/extra/gc_cpp.h
new file mode 100644
index 0000000..36669f9
--- /dev/null
+++ b/include/extra/gc_cpp.h
@@ -0,0 +1,2 @@
+/* This file is installed for backward compatibility. */
+#include <gc/gc_cpp.h>
diff --git a/include/include.am b/include/include.am
index 78c57c3..6b82b0f 100644
--- a/include/include.am
+++ b/include/include.am
@@ -52,3 +52,8 @@ dist_noinst_HEADERS += \
        include/cord.h \
        include/ec.h \
        include/javaxfc.h 
+
+# unprefixed header
+include_HEADERS += \
+       include/extra/gc.h \
+       include/extra/gc_cpp.h



More information about the Gc mailing list