[PATCH] Python bindings (with much of the core api)

[Mark Jenkins]

Derek Atkins wrote:
> new optional directory you also need to add it to DIST_SUBDIRS
> otherwise "make dist" (or distcheck) will fail if you configured the
> to-be-disted tree without your optional directory enabled.
> DIST_SUBDIRS completely overrides SUBDIRS so if you have DIST_SUBDIRS
> then you need to include all the subdirs in it (optional or
> otherwise).

Thanks for catching that.

If anyone is interested, this is the relevant page in the Automake manual:
http://www.gnu.org/software/libtool/manual/automake/Conditional-Subdirectories.html

Attached is a new version of the patch, with this one aspect fixed.

Derek also wrote:
> Quick question: Is there any reason you're duplicating the swig
> wrapping from existing .i files?  Can you not use the same interface
> files for guile and python?

We have some parts (in src/optional/python-bindings/glib.i and
src/optional/python-bindings/timespec.i) that do SWIG type mapping
specifically for python; they make calls to the python C api.

I can see why src/optional/python-bindings/gnucash_core.i is possibly
quite redundant though.

I hate redundancy just as much as the next developer - I hope Jeff and I
can put some thought and investigation into how we can deal with this.


Mark
-------------- next part --------------
A non-text attachment was scrubbed...
Name: python-bindings_2008-03-24.patch
Type: text/x-patch
Size: 43208 bytes
Desc: not available
Url : http://lists.gnucash.org/pipermail/gnucash-devel/attachments/20080324/1fb4f12c/attachment-0001.bin