-
-
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
add librealsense2 packages #230
Conversation
ouch, getting hit by a mamba bug :) |
let me make release and we retry in a couple hours. sorry |
Haha no worries! |
And regarding using |
Release for mamba in the pipeline, but it'll take ~3 hours to propagate conda-forge/mamba-feedstock#146 |
I haven't personally tried that package yet but I trust @traversaro a lot :) I am sure if you guys want you can become co-maintainers of the conda-forge feedstock (https://github.com/conda-forge/librealsense-feedstock) to make sure that updates are merged quickly etc. |
Sure! Feel free to open a PR in https://github.com/conda-forge/librealsense-feedstock/pulls if you like. The general problem of how to handle libraries that are both in ROS buildfarm and in conda-forge is discussed in RoboStack/robostack.github.io#17, and we still do not have a perfect solution, especially for packages that are not marked as |
The Windows builds fail @baszalmstra - would be great if you could have a look (https://dev.azure.com/robostack/ros_pipelines/_build/results?buildId=3522&view=logs&j=2a05d173-1a26-5655-a01b-1fc4742df1ca&t=68e8f140-1776-5e5d-5257-4d245990caf9). |
See also #131 |
@Tobias-Fischer I think we can simply remove it again from Windows. It was my fault, I merged too quickly |
I will make a PR to remove it from windows! |
Here you go: #233 |
Adds the
librealsense2
,realsense2-camera
, andrealsense2-description
packages.