Hi all,

Eigen2 is an optional dependency of OpenBabel upon which a small
number of new classes depend.

I can add #ifdef HAVE_EIGEN2 to openbabel-python.i, and this will work
fine to generate openbabel-python.cpp. However, the ultimate end-user
may not have Eigen2 and so when she compiles openbabel-python.cpp she
will run into trouble if the release manager had Eigen2 installed and
she doesn't.

So...I recommend that the release manager does not have HAVE_EIGEN2
defined. This way the distributed language bindings will omit classes
that require Eigen2. If they want to use these classes (which
currently are just a small number), they will need to run swig
themselves (which I think is reasonable).

To test, check whether OBEnergyConformerScore exists in openbabel.py.
If it does, then the end-user will require Eigen2 to use the Python
bindings.

- Noel

------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
of $1 Million in cash or HP Products. Visit us here for more details:
http://ad.doubleclick.net/clk;226879339;13503038;l?
http://clk.atdmt.com/CRS/go/247765532/direct/01/
_______________________________________________
OpenBabel-Devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openbabel-devel

Reply via email to