-
Notifications
You must be signed in to change notification settings - Fork 38
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
Fix python bindings compilation when toml is not installed #700
Conversation
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.
Thanks!
@@ -1,6 +1,10 @@ | |||
# Changelog | |||
All notable changes to this project are documented in this file. | |||
|
|||
## [unreleased] | |||
### Fixed | |||
- Fix python bindings compilation when toml is not installed (https://github.com/ami-iit/bipedal-locomotion-framework/pull/700) |
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.
We never released a bugged version, so I am not sure if you want to include this in the CHANGELOG.
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.
the version 0.14.0 contains the bug :(
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.
bipedal-locomotion-framework/bindings/python/bipedal_locomotion_framework.cpp.in
Lines 124 to 126 in 2736a04
@cmakeiftarget BipedalLocomotion::ParametersHandler && BipedalLocomotion::ParametersHandlerYarpImplementation | |
bindings::ParametersHandler::CreateTomlModule(parametersHandlerModule); | |
@endcmakeiftarget BipedalLocomotion::ParametersHandler && BipedalLocomotion::ParametersHandlerYarpImplementation |
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.
Ah right I was confusing this change with the CMake files change.
Introduced by #698