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

Install conda #73

Open
wants to merge 4 commits into
base: latest
Choose a base branch
from

Conversation

ahcorde
Copy link

@ahcorde ahcorde commented Jun 17, 2024

This PR install conda the same way Python is installed with Python.rb.

It builds on top of this other PR #72

Signed-off-by: Alejandro Hernandez Cordero <[email protected]>
Signed-off-by: Alejandro Hernández Cordero <[email protected]>
@claraberendsen
Copy link
Contributor

This looks like it's on the right path. You would need to add the recipe like the other ones in

include_recipe 'ros2_windows::python'

As I understand the conda_packages are to manage the installation of the dependencies through conda no? I would prefer if we can have one recipe that installs conda and another one that manages the conda dependencies (it should work as is either way, more of an stylistic choice)

@traversaro
Copy link

If you want to install packages with conda-forge, probably you may want to use an installer that uses conda-forge by default like miniforge3? The anaconda installer uses the defaults channel that has quite restrictive constraint, and for some companies you can't use it free of charge, see https://www.anaconda.com/blog/is-conda-free .

Signed-off-by: Alejandro Hernández Cordero <[email protected]>
Signed-off-by: Alejandro Hernández Cordero <[email protected]>
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