Re: [gnome-db] mergeant cvs compile error

[Rodrigo Moya <rodrigo gnome-db org>]

On Mon, 2003-05-12 at 03:05, Gonzalo Paniagua Javier wrote:
> El lun, 12 de 05 de 2003 a las 02:21, Mike escribió:
> > current make of mergeant bombs as follows
> > 
> > conf-manager.c:157: parse error before '*' token
> [...]
> > conf-manager.c:239: `GConfClientNotifyFunc' undeclared (first use in
> > this function)
> > conf-manager.c:239: parse error before "config_notification_cb"
> 
> I've just built mergeant from CVS.
> 
> That looks like gconf header files are not included.
>
yes, with the changes made to libgnomedb to compile some parts depending
on whether gnome support should be built make this happen. We have an
ugly #ifdef BUILD_WITH_GNOME in the public headers.

I guess we should remove that and maybe generate libgnomedb.h
automatically, depending on the gnome support. Having a #ifdef in a
public header is not a very good idea, since it makes apps have the need
to -D that define.

I fixed it in mergeant by including gnome-db-config.h instead of
libgnomedb.h.

cheers