-
-
Notifications
You must be signed in to change notification settings - Fork 13
Cleanup pixi.toml, consolidate vinca.yaml in a single file and cleanup build of livox-ros-driver2 #79
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
Conversation
693b302
to
86923ff
Compare
That’s amazing, thanks so much @traversaro :)! I wonder if one day we can take it a step further and unify most of the files across distributions … Do you know why it rebuilds files in CI in the PR? The Windows build is failing unfortunately. |
…_recipes instead of manually mantained additional recipes
617441c
to
ca48f0d
Compare
Yes, I think in the merge I started building
That's the goal, I think that if we can simplify the amount of files in the different distro that becomes also easier. |
I did an upstream fix in OctoMap/octomap_rviz_plugins#50, but then the build fails with:
that I think it is actually due to some missing symbol export in rviz, so I think we can just skip for now. |
Ok, skipped in a7edd5c that is also a good check if the whole selectors stuff works. |
Anyhow, just as an investment for the future, I fixed the root issue in rviz: ros2/rviz#1492 . |
@Tobias-Fischer CI is happy. |
The new vinca file looks a lot cleaner. And it's more clearly laid out why some aren't built on windows. Platform parity will be a lot more maintainable now. I'll make sure to update my apriltag PR. |
This PR contains several cleanups that hopefully will make the project more maintainable:
default
andbeta
environments of the pixi.toml are unified in a singledefault
environments, a almost always we were using thebeta
environment, so there is not so much sense for a separatebeta
environmentvinca_*.yaml
files have been consolidated in a singlevinca.yaml
file, that use rattler-build selectors to include or exclude specific packages. See Add ratter-build style selector for vinca.yaml, dependencies.yaml and pkg_additional_info.yaml vinca#77 for the related PR in vinca.rosdistro_additional_recipes.yaml
mechanism introduced in Remove the need for most manually managed additional_recipes vinca#78 instead of manually maintaining the corresponding recipe