26.07.10, 14:22, "Noel O'Boyle" <[email protected]>:
> 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). I see at least two other ways to resolve this: 1) make Eigen2 obligatory 2) check for Eigen2-related classes at run time (generate some placeholder wrappers for classes which use Eigen) Just my $0.02 -- Regards, Konstantin ------------------------------------------------------------------------------ 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
