-
-
Notifications
You must be signed in to change notification settings - Fork 72
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
Full Rebuild January 2024 with updated conda-forge pinnings #418
Conversation
linux-aarch64 failed early with error:
|
Remove changes upstreamed in moveit/moveit#2772
Thanks for taking the initiative, @traversaro! Regarding pinnings, some more candidates:
I'll probably have time to help out in a few weeks time once the semester is over. I found that it is fastest to iterate by locally building first, then try building in CI. How about a bump of Python as well? At least to 3.11, potentially even 3.12? |
Sure! I was just confused on how to replicate the CI locally, but after following the steps in the CI scripts locally, now I was able to replicate them.
I am not sure about this, at the moment one of my main goals were fixing a few problems with Python 3.9 packages, once the build works fine probably we can think if either get a full rebuild with Python 3.9 packages, and after update, or how to proceed. |
ffmpeg=6 is already done in the updated conda_forge pinnings, for the others I can add them in the local config! |
CI builds fail with:
Somehow, the updated additional package for ros-noetic-eigenpy is not build. Manually building locally make the build go a lot forward, even if there are build failures for hpp-fcl and pinocchio. To solve them, I also converted them to additional packages that only depend on the conda-forge versions of the packages. Probably it would be a good time to work on a slightly better solution for RoboStack/robostack.github.io#17 . |
macos builds are failing with:
Windows builds are failing with:
I tried to reproduce the Windows failure locally without success. |
These segfaults might be from memory leaks or similar in boa. They often go away in the actual CI on main |
linux-aarch64 fails with:
That as far as I remember was a issue we had also in the past, and we solved by locally building a package. linux-64 fails with:
I solved this locally by manually building the |
On the main branch, we have a native aarch64 build via https://github.com/RoboStack/ros-noetic/blob/master/.cirun.yml so the error should go away. For linux-64, I temporarily enabled building the |
I think run_exports does not work as intended with boa (building ros-noetic-eigenpy fails):
|
@traversaro @wolfv - any chance to give this a quick review, please? |
Everything seems fine, I just have a comment inline but non blocking. |
Just a test for now, but it should solve several failures that we recently saw due to conflicts with conda-forge abi migrations, while robostack package still use older version of the dependencies.
I used the pinning used as of 2023/10/14 in conda-forge/conda-forge-pinning-feedstock@932c544, except for libopencv and bullet-cpp that I manually bumped as they both have abi migration that are almost complete.
Fix #366 .
Fix #415 .