-
Notifications
You must be signed in to change notification settings - Fork 48
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
Add pybind11 dependency to enable installation of manifpy and bipedal-locomotion-framework Python bindings if ROBOTOLOGY_USES_PYTHON is enabled #800
Conversation
To test this, I also launched a dry-run of the |
A lot of build are failing with error:
This is surprising, as |
Windows build are failing with:
As windows build of manifpy are also failing on conda-forge (see conda-forge/manif-feedstock#7), for the time being let's merge the initial support for manifpy on Linux and macOS and work on Windows support in a second moment. |
The generation of conda packages is failing on this branch, with error (on Linux):
|
macOS fails with a similar problem, while Windows fail with:
|
This should be fixed by ce62560 . |
It seems like boa is trying to install iDynTree with Python 3.6, even if Python 3.9 installation is required. |
Switching to
Instead, macOS is still failing for not finding deps:
|
Fix #799 . As it is not trivial to add patches on top of superbuild projects, for now we will rely on https://github.com/robotology-dependencies/manif/releases/tag/0.0.4.1 and we will go back to upstream artivis on next release.
As
manifpy
is now available as a separate package in conda-forge for Linux and macOS (see conda-forge/manif-feedstock#4), I also added that as a conda-forge dependency for bipedal-locomotion-framework (see https://github.com/robotology/robotology-superbuild/blob/master/doc/developers-faqs.md#how-to-ensure-that-binary-packages-are-correctly-generated-for-a-new-package for background).For the time being several problems were found in the 0.2.0 blf package w.r.t. conda binary package generation, so in this PR the generation of that package is disabled (see #806).