Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enabling Eigen in travis #1214

Merged
merged 130 commits into from
Nov 18, 2016
Merged

Enabling Eigen in travis #1214

merged 130 commits into from
Nov 18, 2016

Conversation

dcoeurjo
Copy link
Member

@dcoeurjo dcoeurjo commented Nov 1, 2016

PR Description

Update of the travis scripts to enable Eigen tests. This would enables unit-test for the DEC package for instance.

Checklist

  • Unit-test of your feature with Catch.
  • Doxygen documentation of the code completed (classes, methods, types, members...)
  • Documentation module page added or updated.
  • New entry in the ChangeLog.md added.
  • No warning raised in Debug cmake mode (otherwise, Travis C.I. will fail).
  • All continuous integration tests pass (Travis & appveyor)

rolanddenis and others added 30 commits August 31, 2016 11:54
@dcoeurjo
Copy link
Member Author

@kerautret , @elcerdo I have refactorized travis script (with child scripts in .travis/ folder). It is clearer now and the Eigen install seems to be ok now (version 3.2.10).

@elcerdo can you please have a look to the DEC issues with this eigen ?

On mac, there is an issue with libqglviewer but it is not related to Eigen issues.

@dcoeurjo
Copy link
Member Author

ping @elcerdo

@elcerdo
Copy link
Contributor

elcerdo commented Nov 17, 2016

DEC tests seams to be fixed. Remaining Travis failures are caused by compilation/cmake errors.
FYI tests failed because there has been a change in boost::hash_range between the antiquated Travis boost version and reasonably recent desktop boost version.

@dcoeurjo
Copy link
Member Author

Ok. Travis is fine again.

@dcoeurjo
Copy link
Member Author

mreging

@dcoeurjo dcoeurjo closed this Nov 18, 2016
@dcoeurjo dcoeurjo reopened this Nov 18, 2016
@dcoeurjo dcoeurjo merged commit f83fa3c into DGtal-team:master Nov 18, 2016
@dcoeurjo dcoeurjo deleted the EigenTravis branch October 7, 2021 11:58
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.

4 participants