Skip to content
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

pixi: Cleanup handling of dependencies built from source and showcase pixi+colcon integration to install dependencies from source #184

Merged
merged 1 commit into from
Mar 7, 2024

Conversation

traversaro
Copy link
Member

Inspired by a comment by @S-Dafarra at the latest "Research Methods and Tools", I realized that I was being a bit sloppy in handling the installation of dependencies from source in the pixi workflow I added in this repo. Instead, it was a perfect occasion of how pixi and vcstool/colcon can work together, pixi handling binary dependencies, while vcstool/colcon handling (in a bit more manual way) the dependencies installed from source.

The download_deps and install_deps steps drastically simplifying by using vcstool and colcon (the tools typically used in ROS 2 workflow). I feel that maintenance is also greatly improved in this way, as if one needs to add a source dependencies it is just needed to add an entry in the pixi_source_deps.yaml file, and optionally add the CMake options used in pixi_source_deps_options.meta.

To keep the nice pattern of using pixi_ prefix for pixi-related files, I renamed the set_install_prefix.* script to pixi_activation.*, that also reflects better what they do as I integrated as I integrated setting the CMAKE_PREFIX_PATH and PYTHONPATH env variables that as @martinaxgloria discovered in robotology/icub-tests#71 can be used to ensure isolation w.r.t. robotology-superbuild's setup.sh sourced in the terminal.

The final result is that there are a lot of pixi_* files hanging around in the root directory of the repo. I am unsure if it make sense to move them in a dedicated directory, for now let's leave them in the root directory of the repo.

@traversaro traversaro changed the title pixi: Cleanup handling of dependencies built from source pixi: Cleanup handling of dependencies built from source and showcase pixi+colcon integration Mar 6, 2024
@traversaro traversaro changed the title pixi: Cleanup handling of dependencies built from source and showcase pixi+colcon integration pixi: Cleanup handling of dependencies built from source and showcase pixi+colcon integration to install dependencies from source Mar 6, 2024
@traversaro traversaro merged commit 6e088aa into master Mar 7, 2024
9 checks passed
@HosameldinMohamed HosameldinMohamed deleted the cleanuppixisrc branch March 7, 2024 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants