2010/7/15 Konstantin Tokarev <[email protected]>: > > > 14.07.10, 22:51, "Geoffrey Hutchison" <[email protected]>: > >> >> On Jul 14, 2010, at 12:13 PM, Konstantin Tokarev wrote: >> > Current version of OB cannot be built without eigen (due to recent >> changes in align.cpp) >> > Is Eigen now mandatory, or ifdefs are missing? >> >> Fixed, thanks. Eigen is *not* required for Open Babel 2.x -- we're not >> going to add more dependencies. > > Not fixed, still can't build without eigen. OBAlign object is used in > conformersearch.cpp, so either it may be #ifdefed in that file, or > placeholder for OBAlign should be provided if Eigen is missing. > > Another problem is that a *lot* of Linux distributions don't include > <Eigen/Dense> header into packages (among them Ubuntu, OpenSuse, Mandriva), > so its presence should be tested.
The problem was that a bug in our own Cmake code prevented this header from being installed until eigen 2.0.12. It's been fixed in 2.0.13. Indeed, that's recent, so a lot of distributions don't have it. Benoit > -- > Regards, > Konstantin > > ------------------------------------------------------------------------------ > This SF.net email is sponsored by Sprint > What will you do first with EVO, the first 4G phone? > Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first > _______________________________________________ > OpenBabel-Devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/openbabel-devel > ------------------------------------------------------------------------------ This SF.net email is sponsored by Sprint What will you do first with EVO, the first 4G phone? Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first _______________________________________________ OpenBabel-Devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openbabel-devel
