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

Boehm, Hans hans.boehm at hp.com
Wed Aug 29 11:39:20 PDT 2007


Thanks.  I put this in my tree (after some reformatting of the stup
include fails, which seem to have gotten mangled below).  To be checked
in shortly.

I currently didn't put in the warning, since I'm not sure there is a
plan to get rid of these.

Hans

> -----Original Message-----
> From: gc-bounces at napali.hpl.hp.com 
> [mailto:gc-bounces at napali.hpl.hp.com] On Behalf Of Petter Urkedal
> Sent: Sunday, August 26, 2007 2:33 PM
> To: gc at napali.hpl.hp.com
> Subject: Re: [Gc] bdw-gc.pc: Cflags: -I${includedir}/pc ?
> 
> 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
> 
> _______________________________________________
> Gc mailing list
> Gc at linux.hpl.hp.com
> http://www.hpl.hp.com/hosted/linux/mail-archives/gc/
> 



More information about the Gc mailing list