search

cryos / avogadro

Avogadro 1 is not under active development, the repository was archived in September 2021. Development of Avogadro 2 is being done at https://github.com/openchemistry/avogadrolibs. Avogadro is an advanced molecular editor designed for cross-platform use in computational chemistry, molecular modeling, bioinformatics, materials science, and related areas.
http://avogadro.cc/
GNU General Public License v2.0
332 stars 155 forks source link

error: Eigen2-support is only available up to version 3.2. Please go to "http://eigen.tuxfamily.org/index.php?title=Eigen2" for further information #915

Open yurivict opened 5 years ago

yurivict commented 5 years ago

On FreeBSD the build for the version 1.2.0 fails:

In file included from libavogadro/src/extensions/surfaces/openqube/atom.cpp:17:
In file included from libavogadro/src/extensions/surfaces/openqube/atom.h:22:
In file included from libavogadro/src/extensions/surfaces/openqube/molecule.h:27:
/usr/local/include/eigen3/Eigen/Core:340:2: error: Eigen2-support is only available up to version 3.2. Please go to "http://eigen.tuxfamily.org/index.php?title=Eigen2" for further information
#error Eigen2-support is only available up to version 3.2. Please go to "http://eigen.tuxfamily.org/index.php?title=Eigen2" for further information
 ^
1 error generated.

eigen-3.3.5 is installed.

JackPerdue commented 4 years ago

This should be noted in the INSTALL file. At present it allows 2.x and 3.x. It should note that it is 2.x and 3.x < 3.3. Not having to rebuild OpenBabel with < 3.3 will save some people time and hopefully removes the error above.

jack (off to rebuld OpenBabel)

JackPerdue commented 4 years ago

I should also note that OpenBabel 3.x doesn't seem to work either (tried 3.0.0). It fails during configure/cmake, whereas the Eigen issue doesn't show up until make.

ghutchis commented 4 years ago

We're not maintaining the old codebase anymore, which we thought was clear from the Github notice and lack of commits.

JackPerdue commented 4 years ago

It wasn't (clear). You could be more explicit like "this is the old version that is no longer maintained and what you really want is this link for new version".

You could also add it to the top of the README... in GREAT BIG LETTERS to make it clear/obvious.

The lack of commits was certainly a sign.

Thanks!!! for your reply... I guess I'm off to build v2.