On dinsdag 8 november 2011, Bill Nottingham wrote: > Geert Janssens (janssens-ge...@telenet.be) said: > > On vrijdag 13 mei 2011, Bill Nottingham wrote: > > > I was doing some testing with guile-2.0 and GnuCash, and encountered > > > the need for the following fixes to make it build. > > > > > > (Note: it still doesn't *work*, but it at least builds for further > > > testing.) > > > > > > Total changes: > > > configure.ac | 14 ++++++++------ > > > src/import-export/aqbanking/Makefile.am | 1 + > > > src/plugins/bi_import/Makefile.am | 1 + > > > 3 files changed, 10 insertions(+), 6 deletions(-) > > > > > > _______________________________________________ > > > gnucash-devel mailing list > > > gnucash-devel@gnucash.org > > > https://lists.gnucash.org/mailman/listinfo/gnucash-devel > > > > Bill, > > > > I just discovered this series of your patches in my mailbox, all covered > > with dust ;) > > > > I'm under the impression they never got applied, right ? > > AFAIK, yes, although I've only really checked on the 2.4 branch. > > > Patch 2 is obsolete by now [1]. I have made some other changes in the > > source so aqbanking and bi_import are no longer dependent on guile. > > > > Patch 3 seems to make sense to me, I'll apply it shortly. > > > > And regarding patch 1, I'm a bit confused. While working on the bug > > mentioned below, Petr had written a patch very similar to your patch 2, > > yet it did seem to work without applying patch 1. So your patch series > > suggests GUILE_INCS has to be AC_SUBST before it can be included, but > > Petr's patch suggests otherwise. Is GUILE_INCS automatically substituted > > as part of the other macro's perhaps ? > > > > I'm not that well versed in the autotools magic, so I prefer to ask > > first. > > Hah, neither am I. But looking at it, GUILE_INCS is in most of the > Makefile.am files, so having an explicit substitute of it looks right. (The > problem I remember having is it not finding guile correctly in > /usr/include/guile/2.0/... which is where our test guile-2.0 packages were > putting it). > Ok, I have added the patch for AC_SUBST(GUILE_INCS) as well. If that's wrong, somebody will surely complain.
Obviously, both patches are only applied to the development tree, since it is not planned to get the 2.4 series to work with guile 2.0. Geert _______________________________________________ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel