Skip to content

Eigen: also copy signature file and don't copy CMake files#709

Merged
boegel merged 1 commit intoeasybuilders:developfrom
torbjoernk:feature/eigen
Oct 8, 2015
Merged

Eigen: also copy signature file and don't copy CMake files#709
boegel merged 1 commit intoeasybuilders:developfrom
torbjoernk:feature/eigen

Conversation

@torbjoernk
Copy link
Copy Markdown
Contributor

The signature file is used by CMake's FindEigen module to determine the
include path of Eigen3.

The CMakeLists.txt files are only used by Eigen3's CMake based installation.

The signature file is used by CMake's FindEigen module to determine the
include path of Eigen3.

The CMakeLists.txt files are only used by Eigen3's CMake based installation.

Signed-off-by: Torbjörn Klatt <[email protected]>
@hpcugentbot
Copy link
Copy Markdown

Automatic reply from Jenkins: Can I test this?

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 8, 2015

Jenkins: ok to test

@hpcugentbot
Copy link
Copy Markdown

Easyblocks unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/1259/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 8, 2015

Looks great, tested with all existing Eigen easyconfigs, works like a charm, so going in.

Thanks @torbjoernk!

boegel added a commit that referenced this pull request Oct 8, 2015
Eigen: also copy signature file and don't copy CMake files
@boegel boegel merged commit b23a7ef into easybuilders:develop Oct 8, 2015
@torbjoernk torbjoernk deleted the feature/eigen branch October 9, 2015 08:26
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.

4 participants