Skip to content

make Eigen a build dependency for nanopolish#8448

Merged
vanzod merged 1 commit intoeasybuilders:developfrom
boegel:nanopolish_Eigen
Jun 6, 2019
Merged

make Eigen a build dependency for nanopolish#8448
vanzod merged 1 commit intoeasybuilders:developfrom
boegel:nanopolish_Eigen

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Jun 6, 2019

Eigen is just a bunch of header files, so there's little point in making it a runtime dep...

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Jun 6, 2019

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node3126.skitty.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 2.7.5
See https://gist.github.com/f07ba5dc62b9df65948566a8d57d1bb8 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Jun 6, 2019

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2446.golett.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/d15768707e3f26956f30825f050c2c82 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Jun 6, 2019

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2003.delcatty.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/5fd5d3a254aade7ab53cd27cd7794974 for a full test report.

@vanzod
Copy link
Copy Markdown
Member

vanzod commented Jun 6, 2019

Test report by @vanzod
SUCCESS
Build succeeded for 4 out of 4 (2 easyconfigs in this PR)
cermis - Linux debian 9.9, Intel(R) Xeon(R) CPU X5670 @ 2.93GHz, Python 2.7.13
See https://gist.github.com/47110e12b3c06eb83b46e7897680a25e for a full test report.

@vanzod
Copy link
Copy Markdown
Member

vanzod commented Jun 6, 2019

Going in, thanks @boegel!

@vanzod vanzod merged commit f880373 into easybuilders:develop Jun 6, 2019
@boegel boegel deleted the nanopolish_Eigen branch June 6, 2019 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants