-
Notifications
You must be signed in to change notification settings - Fork 9
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
Initial conda configurations import (pixi testing on github actions) #1192
Conversation
Signed-off-by: Jose Luis Rivero <[email protected]>
* Add github action testing --------- Signed-off-by: Jose Luis Rivero <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this file necessary if we have the pixi.toml
file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is not in use by us. It was the first file designed to work with the other mamba/micromamba/conda package managers. We could just remove it if we are not going to use it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd rather avoid duplicating the information unless it's truly necessary.
Signed-off-by: Jose Luis Rivero <[email protected]>
conda/envs/legacy/pixi.toml
Outdated
boost-cpp = "1.78.0.*" | ||
bullet-cpp = "*" | ||
cppzmq = "*" | ||
cuda = "*" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CUDA is not necessary
conda/envs/legacy/pixi.toml
Outdated
|
||
[dependencies] | ||
assimp = "*" | ||
boost-cpp = "1.78.0.*" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think any Gazebo library depends on Boost explicitly, so this can be removed.
conda/envs/legacy/pixi.toml
Outdated
glib = "*" | ||
gts = "*" | ||
jsoncpp = "*" | ||
libccd-double = "*" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think any Gazebo library depends on libccd-double explicitly, so this can be removed.
conda/envs/legacy/pixi.toml
Outdated
curl = "*" | ||
dartsim = "6.13.2.*" | ||
eigen = "*" | ||
fcl = "0.7.0.*" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think any Gazebo library depends on fcl explicitly, so this can be removed.
Signed-off-by: Jose Luis Rivero <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>
The PR adds the initial conda/pixi configurations together with a README explaining the directory structure and the files contained.