Avogadro 1.1.1: Building with Eigen3 fails

Hi there,

I’m speaking as part of the Debian package maintainers of avogadro. The
latest release 1.1.1 fails to build with Eigen3. The reason is, that
Eigen2 is still required, e.g. in
libavogadro/src/extensions/surfaces/openqube/CMakeLists.txt. Also
cmake/modules/AvogadroUse.cmake requires Eigen2. So it is not possible
to build with Eigen3 only and the build fails. Is this some kind of
intentional result or a bug?

I’m attaching the build log.

Regards, Daniel

On Tue, Dec 31, 2013 at 8:55 AM, Daniel Leidert daniel.leidert@wgdd.de wrote:

Hi there,

I’m speaking as part of the Debian package maintainers of avogadro. The
latest release 1.1.1 fails to build with Eigen3. The reason is, that
Eigen2 is still required, e.g. in
libavogadro/src/extensions/surfaces/openqube/CMakeLists.txt. Also
cmake/modules/AvogadroUse.cmake requires Eigen2. So it is not possible
to build with Eigen3 only and the build fails. Is this some kind of
intentional result or a bug?

I’m attaching the build log.

It is a bug, Geoff submitted a patch to enable building with Eigen 3
for 1.1.1 (this is very new, and not well-tested - thanks for the
feedback). We can work on a fix for 1.1.2 (a few other things have
come up too), I will try and find some time to look into this further
unless Geoff gets to it first.

Marcus

Argh.

Sorry for the very delayed reply. I have a new patch that will go up on Gerrit shortly that should work with Eigen3-only or Eigen-2 only. I had both on my laptop and didn’t catch the bug. :frowning:

Marcus… part of the problem seems to be that the #define in config.h is not getting to all of the code. Indeed, config.h isn’t #imported to much of the Avogadro codebase.

Thanks,
-Geoff