Skip to content

BUG: Update Eigen3 to fix INSTALL_DIR problems#588

Merged
phcerdan merged 2 commits into
InsightSoftwareConsortium:masterfrom
phcerdan:update_eigen3
Mar 14, 2019
Merged

BUG: Update Eigen3 to fix INSTALL_DIR problems#588
phcerdan merged 2 commits into
InsightSoftwareConsortium:masterfrom
phcerdan:update_eigen3

Conversation

@phcerdan

Copy link
Copy Markdown
Contributor
git shortlog --no-merges  0805e8fd8..8a3c2d91

Pablo Hernandez-Cerdan (1):
      BUG: ITK: CMake files are now installed in ITK_INSTALL_PACKAGE_DIR/Modules

Includes patch:

BUG: ITK: CMake files are now installed in ITK_INSTALL_PACKAGE_DIR/Modules

Instead of hard coded path including the version.

When using ITK_INSTALL_PACKAGE_DIR=lib/cmake/ITK
Before (wrong):

]$ find ITK-prefix | grep Eigen3
ITK-prefix/lib/cmake/ITK-5.0/Modules/Eigen3Config.cmake
ITK-prefix/lib/cmake/ITK-5.0/Modules/Eigen3ConfigVersion.cmake
ITK-prefix/lib/cmake/ITK-5.0/Modules/ITKInternalEigen3Config.cmake
ITK-prefix/lib/cmake/ITK-5.0/Modules/ITKInternalEigen3Targets.cmake
ITK-prefix/lib/cmake/ITK-5.0/Modules/ITKInternalEigen3ConfigVersion.cmake
ITK-prefix/lib/cmake/ITK-5.0/Modules/Eigen3Targets.cmake
ITK-prefix/lib/cmake/ITK/Modules/ITKEigen3.cmake

After this fix:

ITK-prefix/lib/cmake/ITK/Modules/Eigen3Config.cmake
ITK-prefix/lib/cmake/ITK/Modules/Eigen3ConfigVersion.cmake
ITK-prefix/lib/cmake/ITK/Modules/ITKInternalEigen3Config.cmake
ITK-prefix/lib/cmake/ITK/Modules/ITKInternalEigen3Targets.cmake
ITK-prefix/lib/cmake/ITK/Modules/ITKInternalEigen3ConfigVersion.cmake
ITK-prefix/lib/cmake/ITK/Modules/Eigen3Targets.cmake
ITK-prefix/lib/cmake/ITK/Modules/ITKEigen3.cmake

Fix #584

kwrobot and others added 2 commits March 14, 2019 14:22
Code extracted from:

    https://gitlab.kitware.com/phcerdan/eigen.git

at commit 8a3c2d91e9d8d70ba187618265869b51610a1f00 (for/itk).

Change-Id: I90eb721c21edeef84510c76d3a100029dbaaacda
* upstream-Eigen3:
  Eigen3 2019-03-14 (8a3c2d91)
@phcerdan phcerdan requested review from blowekamp and thewtex March 14, 2019 18:32

@blowekamp blowekamp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@phcerdan phcerdan merged commit 16d8b5a into InsightSoftwareConsortium:master Mar 14, 2019
@phcerdan phcerdan deleted the update_eigen3 branch March 14, 2019 21:48
hjmjohnson pushed a commit to hjmjohnson/ITK that referenced this pull request May 6, 2026
…_eigen3

BUG: Update Eigen3 to fix INSTALL_DIR problems
hjmjohnson pushed a commit to hjmjohnson/ITK that referenced this pull request May 12, 2026
…_eigen3

BUG: Update Eigen3 to fix INSTALL_DIR problems
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Eigen not installed correctly, unable to build against installed ITK

4 participants