On Friday 12 April 2013 21:07:50 Christian Stimming wrote:
> Am Freitag, 12. April 2013, 09:22:06 schrieb Norbert Holze:
> > Am 11.04.2013 22:39, schrieb Christian Stimming:
> > > Can you add $(GLIB_LIBS) not (only) to LDADD, but rather to
> > > test_sqlbe_LDADD and check again? Christian
> > 
> > It works with:
> > 
> > src/backend/sql/test/Makefile.am
> > 
> > #test_sqlbe_HEADERS = \
> > #    $(top_srcdir)/$(MODULEPATH)/gnc-backend-sql.h
> > 
> > test_sqlbe_LDADD = \
> > 
> >     $(top_builddir)/$(MODULEPATH)/libgnc-backend-sql.la \
> >     $(top_builddir)/src/libqof/qof/libgnc-qof.la \
> >     $(top_builddir)/src/test-core/libtest-core.la \
> >     $(GLIB_LIBS)
> > 
> > Next error:
> > 
> > libtool: compile:  gcc -DHAVE_CONFIG_H -I. -I../../.. -I../../../src
> > -I../../../src/core-utils -I../../../src/engine
> > -I../../../src/gnc-module -I../../../src/app-utils
> > -I../../../src/gnome-utils -I../../../src/register/ledger-core
> > -I../../../src/register/register-core -I../../../src/libqof/qof -pthread
> > -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include
> > -pthread -I/usr/include/gconf/2 -I/usr/include/dbus-1.0
> > -I/usr/lib/x86_64-linux-gnu/dbus-1.0/include -I/usr/include/glib-2.0
> > -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -pthread
> > -I/usr/include/libgnomecanvas-2.0 -I/usr/include/pango-1.0
> > -I/usr/include/gail-1.0 -I/usr/include/libart-2.0 -I/usr/include/gtk-2.0
> > -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include
> > -I/usr/include/harfbuzz -I/usr/include/freetype2 -I/usr/include/atk-1.0
> > -I/usr/lib/x86_64-linux-gnu/gtk-2.0/include -I/usr/include/cairo
> > -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/gio-unix-2.0/
> > -I/usr/include/pixman-1 -I/usr/include/libpng12
> > -DG_LOG_DOMAIN=\"gnc.register.gnome\" -Wdeclaration-after-statement -g
> > -O2 -Wall -Wunused -Wmissing-prototypes -Wmissing-declarations
> > -Wno-unused -MT table-gnome.lo -MD -MP -MF .deps/table-gnome.Tpo -c
> > table-gnome.c  -fPIC -DPIC -o .libs/table-gnome.o
> > In file included from table-gnome.c:51:0:
> > ../../../src/guile-mappings.h:20:54: fatal error: libguile.h: No such
> > file or directory
> > compilation terminated.
> > make[5]: *** [table-gnome.lo] Fehler 1
> > make[5]: Verlasse Verzeichnis
> > '/home/norbert/Downloads/gnucash/gnucash-2.5.0/gnucash-2.5.0/src/register/
> > re gister-gnome' make[4]: *** [all-recursive] Fehler 1
> 
> But regardless of guile-2 vs. guile-1.8 the Makefile here is missing the
> GUILE_INCS in CPPFLAGS, which means the compiler probably correctly doesn't
> find <libguile.h>.
> 
> I've fixed this and the other makefile issue in SVN.
> 
Actually although table-gnome.c is including guile-mappings.h, it is not really 
using it.

I personally prefer to keep the guile dependencies limited to as few building 
blocks as possible in the code 
base. So I have committed another way to fix this in r22901 by removing the 
include line from the c file 
and undoing your GUILE_INCS addition.

Regards,

Geert
_______________________________________________
gnucash-devel mailing list
gnucash-devel@gnucash.org
https://lists.gnucash.org/mailman/listinfo/gnucash-devel

Reply via email to