Re: Glom 1.10 soon



On Thu, 2009-03-19 at 11:30 +0100, Murray Cumming wrote:
On Sun, 2009-03-08 at 14:14 +0100, Armin Burgmeier wrote:
As libglom/libglom_config.h I would suggest to remove all non-prefixed
symbols from it, to prevent conflicts with a "config.h" 

Thanks for fixing that, though I don't quite understand how you did it.

However, there are some duplicate defines (the define is in both
config.h and libglom_config.h files) that are causing warnings in svn
trunk, at least when doing distcheck. Could you try to fix that? I guess
that config.h should include libglom_config.h.

The problem is that config.h.in is generated by autoheader, which
generates it by scanning the configure.ac file for all the AC_DEFINEs I
think.

I believe that to fix this, we would need to get rid of autoheader and
maintain the config.h.in ourselves. This means, whenever adding an
AC_DEFINE to configure.ac, we will either need to add it to config.h.in
or to glom/libglom/libglom_config.h.in, depending on whether its
properly prefixed (and meant for installation) or not.

I did this, however, I cannot test it right now since glom trunk does
not build for me because of bug #576069.

Armin




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]