-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
gtsam: Add options new options #17931
Conversation
Add options to set use_system_eigen and with_march_native
|
This comment has been minimized.
This comment has been minimized.
Conan v1 pipeline ❌Sorry, the build is only launched for Access Request users. You can request access writing in this issue. Conan v2 pipeline ❌
The v2 pipeline failed. Please, review the errors and note this will be required for pull requests to be merged in the near future. See details:Sorry, the build is only launched for Access Request users. You can request access writing in this issue. |
@@ -67,6 +70,7 @@ class GtsamConan(ConanFile): | |||
"install_matlab_toolbox": False, | |||
"install_cython_toolbox": False, | |||
"install_cppunitlite": True, | |||
"use_system_eigen": True, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should probably make the self.requires("eigen/3.4.0")
conditional on this being set to True
then as well.
I detected other pull requests that are modifying gtsam/all recipe: This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there. |
Cancelled |
Add options to set use_system_eigen and with_march_native
Specify library name and version: gtsam