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

Do not quietly search for ASSIMP to decide if enable it or not #600

Merged
merged 4 commits into from
Nov 26, 2019

Conversation

traversaro
Copy link
Member

@traversaro traversaro commented Nov 20, 2019

Add DO_NOT_SILENTLY_SEARCH option to avoid the initial find_package QUIET call to decide the initial default value of the option to enable a dependency.

Enable it for Assimp as a workaround for #599 .

Assimp was added to GitHub Actons CI to validate that the solution is working.

Add DO_NOT_SILENTLY_SEARCH option to avoid the initial `find_package` QUIET call to decide the initial default value of the option to enable a dependency.

Enable it for Assimp as a workaround for #599 .
@traversaro traversaro changed the title Add assimp to GitHub Actions CI Do not quietly search for ASSIMP to decide if enable it or not Nov 26, 2019
@traversaro traversaro requested a review from lrapetti November 26, 2019 11:59
Copy link
Member

@lrapetti lrapetti left a comment

Choose a reason for hiding this comment

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

I tested it and I am now able to build iDynTree with assimp5 installed and the flag set to OFF.

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.

2 participants