[Gc] Dependency tracking for configuration macros

Petter Urkedal urkedal at nbi.dk
Sat Sep 12 05:42:46 PDT 2009


On 2009-09-12, Ivan Maidanski wrote:
> Petter Urkedal <urkedal at nbi.dk> wrote:
> > Patch 1
> > 
> > Fix/add inclusion of private/config.h.
> >     * include/private/gc_priv.h, include/private/gc_pmark.h, gc_cpp.cc,
> >     tests/test.c: Change include of config.h to private/config.h.
> >     * tests/test_cpp.cc: Include private/config.h.
> > ...
> 
> Ok (but I think "#undef GC_BUILD" should be added after including
> config.h in test_cpp.cc (as  in test.c)). I'll commit patch 1 in
> several days if Hans don't mind.

Thanks.  I don't think #undef GC_BUILD is necessary on behalf of
config.h, since GC_BUILD is not a configuration macro.  As far as I
understand the use of GC_BUILD, it indicates that the libgc library is
being built, and this is only known by the build system, not by
configure.


More information about the Gc mailing list